This is the mail archive of the mailing list for the GCC project.

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

Re: Basic block profiler out of libgcc

> 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

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

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