[RFA] patch to add 'maint profile-gdb' command

Andrew Cagney ac131313@cygnus.com
Wed Sep 12 11:28:00 GMT 2001


>  i think the command should always be present.  looking at the patch, it 
>> appears to have started out that way.
> 
> 
> I'd disagree.  First, these functions are not very portable.  Our
> very own RH 6.2 box that is sourceware.cygnus.com does not have
> moncontrol() or monstart(), and -pg doesn't seem to be usable at
> all.  My RH 7.1 box at home works fine.  MacOS X's FreeBSD works
> fine.  Second, you definitely don't want to compile gdb with -pg
> by default - the compiler inserts a bunch of bookkeeping code and
> functions, and that'll be a real performance penalty.

while the command is always present it wouldn't do much by default.  the 
function that implements the command throws an error.

is that making more sense?

> are you sure no changes to maint.exp are needed? not needing them feels 
>> wrong.
> 
> 
> The "help maint" test in maint.exp globs it up.  You don't want to include
> a check for maint profile-gdb in here because you'd get a FAIL when gdb
> was not configured --enable-profiling.

or to turn it around, will an --enable-profiling gdb have additional 
failures?  i don't think that enabling profiling should affect the test 
results.

andrew




More information about the Gdb-patches mailing list