This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: Patch: --enable-profiling
- To: tromey at cygnus dot com
- Subject: Re: Patch: --enable-profiling
- From: Stan Shebs <shebs at cygnus dot com>
- Date: Fri, 29 Oct 1999 17:45:28 -0700
- CC: gdb-patches at sourceware dot cygnus dot com
From: Tom Tromey <tromey@cygnus.com>
Date: 29 Oct 1999 14:43:09 -0600
This patch adds a `--enable-profiling' option to gdb's configure
script. If given, this defines ENABLE_PROFILE and builds gdb with
`-pg'. ENABLE_PROFILE adds a new maintenance command which can be
used to enable or disable profiling from the command line. (By
default profiling is turned off in main()).
Ok to check in?
This is cool! (I was going to reply earlier that this was a good idea,
glad you went ahead and finished the patch.)
You should add a note to gdb/NEWS, a mention of --enable-profiling
somewhere in the internals manual (perhaps connected to the section on
coding, but I don't have a strong preference), and a mention in the
main GDB manual, with the rest of the maintenance commands.
Connected with the last, I'd like you to make the command exist
unconditionally, but if --enable-profiling wasn't used, to say
something "This GDB executable was not compiled with -pg" or some
such, rather than having a documented command be mysteriously absent,
with no explanation as to why.
BTW, will this work with djgpp and/or cygwin? What will happen if
you try to configure on those with --enabling-profiling?
Stan