From: Andris Pavenis <pavenis AT latnet DOT lv>
To: djgpp-workers AT delorie DOT com, DJ Delorie <dj AT delorie DOT com>
Subject: Re: [gkeating AT apple DOT com: fix profiling on Darwin]
Date: Mon, 10 Feb 2003 08:39:14 +0200
User-Agent: KMail/1.5
References: <200301312308 DOT h0VN8KP12285 AT envy DOT delorie DOT com>
In-Reply-To: <200301312308.h0VN8KP12285@envy.delorie.com>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200302100839.14545.pavenis@latnet.lv>
Reply-To: djgpp-workers AT delorie DOT com

On Saturday 01 February 2003 01:08, DJ Delorie wrote:
> do we?

This is not needed for as far as I understand (there is no libgcov.a). 3.2.2 
binaries are built, but not yet uploaded and are
	http://hal.astr.lu.lv/pub/djgpp/gcc/3.2.2/
No real testing done (except the bootstrapping of course). Also profiling is 
not tested.

Maybe I'll take a look when build some gcc-3.3 snapshot sometimes. I don't 
know when I'll do that, as I'm using DJGPP very little in last time (I'm 
mostly using Linux now)

Andris

> ------- Start of forwarded message -------
> Date: Fri, 31 Jan 2003 15:07:23 -0800 (PST)
> From: Geoffrey Keating <gkeating AT apple DOT com>
> To: gcc-patches AT gcc DOT gnu DOT org
> CC: Nathan Sidwell  <nathan AT codesourcery DOT com>, dj AT delorie DOT com
> Subject: fix profiling on Darwin
>
>
> This patch:
>
> 2003-01-27  Nathan Sidwell  <nathan AT codesourcery DOT com>
>
> 	* Makefile.in (LIB2FUNCS_ST): Remove _gcov.
> 	(LIBGCOV): New variable.
> 	(libgcc.mk): Add LIBGCOV.
> 	(LIBGCC_DEPS): Add libgcov.c.
> 	(libgcov.a): New target.
> 	(clean): Remove libgcov.a.
> 	(install-libgcc): Do libgcov too.
> 	(stage1-start, stage2-start, stage3-start, stage4-start): Deal
> 	with libgcov.a.
> 	* libgcc2.c (L_gcov): Move into ...
> 	* libgcov.c: ... here. New file.
> 	* mklibgcc.in: Add libgcov rules.
> 	* gcc.c (LINK_COMMAND_SPEC): Add -lgcov when profiling.
>
> broke profiling on Darwin because Darwin has its own
> LINK_COMMAND_SPEC and Nathan didn't update it.  This patch fixes it.
>
> Probably djgpp also needs a similar patch since it also has its own
> LINK_COMMAND_SPEC.
>
> Bootstrapped & tested on powerpc-darwin.
>
> - --
> - - Geoffrey Keating <geoffk AT apple DOT com>
>
> ===File ~/patches/gcc-darwingprof.patch=====================
> 2003-01-31  Geoffrey Keating  <geoffk AT apple DOT com>
>
> 	* config/darwin.h (LINK_COMMAND_SPEC): Update for Nathan's recent
> 	change to LINK_COMMAND_SPEC in gcc.c.
>
> Index: config/darwin.h
> ===================================================================
> RCS file: /cvs/gcc/gcc/gcc/config/darwin.h,v
> retrieving revision 1.47
> diff -u -p -u -p -r1.47 darwin.h
> - --- config/darwin.h	3 Jan 2003 23:09:32 -0000	1.47
> +++ config/darwin.h	31 Jan 2003 23:03:31 -0000
> @@ -190,7 +190,8 @@ Boston, MA 02111-1307, USA.  */
>      %{!Zdynamiclib:%{A} %{e*} %{m} %{N} %{n} %{r} %{u*} %{x} %{z}} \
>      %{@:-o %f%u.out}%{!@:%{o*}%{!o:-o a.out}} \
>      %{!Zdynamiclib:%{!A:%{!nostdlib:%{!nostartfiles:%S}}}} \
> - -    %{L*} %(link_libgcc) %o %{!nostdlib:%{!nodefaultlibs:%G %L}} \
> +    %{L*} %(link_libgcc) %o %{fprofile-arcs:-lgcov} \
> +    %{!nostdlib:%{!nodefaultlibs:%G %L}} \
>      %{!A:%{!nostdlib:%{!nostartfiles:%E}}} %{T*} %{F*} \
>      %{!--help:%{!no-c++filt|c++filt:| c++filt3 }} }}}}}}}}"
>
> ============================================================
> ------- End of forwarded message -------