This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Basic block profiler out of libgcc
- To: zackw at Stanford dot EDU
- Subject: Re: Basic block profiler out of libgcc
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- Date: Sat, 5 May 2001 01:22:39 +0200
- CC: mark at codesourcery dot com, gcc-patches at gcc dot gnu dot org
> Date: Mon, 16 Apr 2001 00:22:33 -0700
> From: "Zack Weinberg" <zackw@Stanford.EDU>
> On Sun, Apr 15, 2001 at 11:20:37AM -0700, Mark Mitchell wrote:
> > - Why remove this comment:
> >
> > -/* Attach a special .ident directive to the end of the file to identify
> > - the version of GCC which compiled this code. The format of the .ident
> > - string is patterned after the ones produced by native SVR4 C
> > -compilers. */
>
> IIRC that was in a lot of config header files, that now have no code
> matching it, so I took out the comment too. I could stick it above
> the block in toplev.c that now emits .ident's.
I see things have changed. Now, if a target does not want a
gcc-specific .ident, it has to be non-intuitive and define
ASM_OUTPUT_IDENT but not IDENT_ASM_OP. Before, it just had to
provide an ASM_FILE_END that did *not* output an .ident for gcc
(compared to config/elfos.h). That does not seem like a good
change.
Again (as I wrote in my patch to *remove* the .ident for gcc),
the .ident for gcc is controlled by flag_no_ident but does not
match the documentation for -fno-ident, which only mentions how
#ident is treated; the documentation does not mention that it
*always* emits one for gcc itself if this option is not used.
The .ident for gcc is not necessary and only has limited
debug-related purposes, while accumulating *for each object* in
executables. It follows the same pattern as gcc2_compiled. I
think it should only be output with -g, if at all.
brgds, H-P