This is the mail archive of the gcc-bugs@gcc.gnu.org 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]
Other format: [Raw text]

[Bug target/33579] INIT_PRIORITY is broken



------- Comment #12 from mark at codesourcery dot com  2007-11-01 16:50 -------
Subject: Re:  INIT_PRIORITY is broken

danglin at gcc dot gnu dot org wrote:
> ------- Comment #11 from danglin at gcc dot gnu dot org  2007-11-01 03:05 -------
> Mark,
> 
> This is major progress.  All the priority tests pass and there are no
> regressions on hppa2.0w-hp-hpux11.11 and hppa-unknown-linux-gnu.
> 
> However, I don't think the patch is quite right.  For example, in the
> gcc.dg/initpri1.c test, two identical routines for "c1" are emitted:

I don't think that's actually a bug -- except maybe its a
misoptimization.  The compiler's just inlining the calls to c1 from the
_GLOBAL_... functions due to code in record_cdtor_fn:

  node->local.disregard_inline_limits = 1;

I'm not sure that's a great idea -- especially with -Os! -- but I also
don't think it's related to the bug I introduced.

Do you agree?

Thanks,


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33579


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