Page MenuHomeFreeBSD

Clean up gprof
ClosedPublic

Authored by andrew on Sep 9 2020, 10:24 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 13, 2:38 AM
Unknown Object (File)
Wed, Nov 13, 2:38 AM
Unknown Object (File)
Wed, Nov 13, 2:38 AM
Unknown Object (File)
Wed, Nov 13, 2:38 AM
Unknown Object (File)
Wed, Nov 13, 2:24 AM
Unknown Object (File)
Wed, Nov 6, 10:11 AM
Unknown Object (File)
Sat, Nov 2, 7:11 PM
Unknown Object (File)
Oct 7 2024, 1:14 PM
Subscribers

Details

Summary
  • Remove identical or almost identical headers
  • List the architectures where aout.c builds rather than where to not build it
  • Enable on all architectures

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33452
Build 30734: arc lint + arc unit

Event Timeline

andrew created this revision.
imp added inline comments.
usr.bin/gprof/Makefile
8

We switched to ELF in FreeBSD 3.2. which means only i386 ever had a.out (though there were prototypes for FreeBSD/alpha with a.out iirc, that's not relevant here).

This should read simply

.if ${MACHINE_CPUARCH} == "i386"

usr.bin/gprof/Makefile
8

<kib> we support i386 a.out on amd64 s/we/I/

Only support aout.c on i386 & amd64

Only support aout.c on i386 & amd64

Crazy question: Do we support building and profiling a.out even at all? Maybe we just drop it entirely?

I have no idea, although if we remove it I think it should be a second change that could be easily reverted if needed.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 14 2020, 4:19 PM
Closed by commit rS365727: Cleanups for gprof: (authored by andrew). · Explain Why
This revision was automatically updated to reflect the committed changes.