This is the mail archive of the gdb-patches@sourceware.cygnus.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Patch: --enable-profiling



   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

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]