bug#3707: add a configure option to enable profiling

bug#3707: add a configure option to enable profiling

[Dan Nicolaescu]

Building emacs with profiling support is not very straight forward at
the moment.
It would be better if we had a configure option to do it:

./configure --enable-profiling
make

Adding the configure flag can be done like this:

--- configure.in.~1.600.~       2009-06-26 18:46:22.000000000 -0700
+++ configure.in                2009-06-28 06:00:37.000000000 -0700
@@ -273,6 +273,18 @@ if test x$ac_gc_check_cons_list != x ; t
 [Define this to check for errors in cons list.])
 fi
 
+AC_ARG_ENABLE(profiling,
+[  --enable-profiling
+                 Build emacs with profiling support],
+[ac_enable_profiling="${enableval}"],[])
+if test x$ac_enable_profiling != x ; then
+   PROFILING_CFLAGS="-DPROFLING -pg"
+   PROFILING_LDFLAGS="-pg"
+else
+   PROFILING_CFLAGS=
+   PROFILING_LDFLAGS=
+fi
+
 #### Make srcdir absolute, if it isn't already.  It's important to
 #### avoid running the path through pwd unnecessarily, since pwd can
 #### give you automounter prefixes, which can go away.  We do all this


How can PROFILING_CFLAGS and PROFILING_LDFLAGS be added to the default
CFLAGS in the Makefile to enable "make" to just DTRT?

bug#3707: add a configure option to enable profiling

[Stefan Monnier]

> Building emacs with profiling support is not very straight forward at
> the moment.
> It would be better if we had a configure option to do it:

> ./configure --enable-profiling
> make

> Adding the configure flag can be done like this:

Looks good, please install,


        Stef

bug#3707: marked as done (add a configure option to enable profiling)

[Emacs bug Tracking System]

[-- Attachment #1: Type: text/plain, Size: 907 bytes --]


Your message dated Fri, 3 Jul 2009 06:20:13 -0700 (PDT)
with message-id <200907031320.n63DKDhg027537@godzilla.ics.uci.edu>
and subject line Re: bug#3707: add a configure option to enable profiling
has caused the Emacs bug report #3707,
regarding add a configure option to enable profiling
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)


-- 
3707: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=3707
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems

[-- Attachment #2: Type: message/rfc822, Size: 3629 bytes --]

From: Dan Nicolaescu <dann@ics.uci.edu>
To: bug-gnu-emacs <bug-gnu-emacs@gnu.org>
Subject: add a configure option to enable profiling
Date: Sun, 28 Jun 2009 14:43:41 -0700 (PDT)
Message-ID: <200906282143.n5SLhf3i014719@godzilla.ics.uci.edu>


Building emacs with profiling support is not very straight forward at
the moment.
It would be better if we had a configure option to do it:

./configure --enable-profiling
make

Adding the configure flag can be done like this:

--- configure.in.~1.600.~       2009-06-26 18:46:22.000000000 -0700
+++ configure.in                2009-06-28 06:00:37.000000000 -0700
@@ -273,6 +273,18 @@ if test x$ac_gc_check_cons_list != x ; t
 [Define this to check for errors in cons list.])
 fi
 
+AC_ARG_ENABLE(profiling,
+[  --enable-profiling
+                 Build emacs with profiling support],
+[ac_enable_profiling="${enableval}"],[])
+if test x$ac_enable_profiling != x ; then
+   PROFILING_CFLAGS="-DPROFLING -pg"
+   PROFILING_LDFLAGS="-pg"
+else
+   PROFILING_CFLAGS=
+   PROFILING_LDFLAGS=
+fi
+
 #### Make srcdir absolute, if it isn't already.  It's important to
 #### avoid running the path through pwd unnecessarily, since pwd can
 #### give you automounter prefixes, which can go away.  We do all this


How can PROFILING_CFLAGS and PROFILING_LDFLAGS be added to the default
CFLAGS in the Makefile to enable "make" to just DTRT?



[-- Attachment #3: Type: message/rfc822, Size: 2329 bytes --]

From: Dan Nicolaescu <dann@ics.uci.edu>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: 3707-done@emacsbugs.donarmstrong.com
Subject: Re: bug#3707: add a configure option to enable profiling
Date: Fri, 3 Jul 2009 06:20:13 -0700 (PDT)
Message-ID: <200907031320.n63DKDhg027537@godzilla.ics.uci.edu>

Stefan Monnier <monnier@iro.umontreal.ca> writes:

  > > Building emacs with profiling support is not very straight forward at
  > > the moment.
  > > It would be better if we had a configure option to do it:
  > 
  > > ./configure --enable-profiling
  > > make
  > 
  > > Adding the configure flag can be done like this:
  > 
  > Looks good, please install,

Thanks.  Done.