[Bug c/31537] duplicate weakref emitted with IMA

2009-06-04 Thread rguenth at gcc dot gnu dot org
--- Comment #13 from rguenth at gcc dot gnu dot org 2009-06-04 13:41 --- We are getting LTO (and maybe LIPO), no need for -combine being fixed. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added

[Bug c/31537] duplicate weakref emitted with IMA

2009-06-04 Thread aldot at gcc dot gnu dot org
--- Comment #12 from aldot at gcc dot gnu dot org 2009-06-04 13:24 --- Well, without it fixed it's impossible to build libgfortran (and other apps) with combine, which would be a very nice thing to have. The sample patch above exposed no regressions fwiw. -- http://gcc.gnu.org/bugzi

[Bug c/31537] duplicate weakref emitted with IMA

2009-06-04 Thread steven at gcc dot gnu dot org
--- Comment #11 from steven at gcc dot gnu dot org 2009-06-04 12:55 --- Oh, the temptation to close this as WONTFIX Objections? -- steven at gcc dot gnu dot org changed: What|Removed |Added -

[Bug c/31537] duplicate weakref emitted with IMA

2008-09-16 Thread aldot at gcc dot gnu dot org
--- Comment #10 from aldot at gcc dot gnu dot org 2008-09-16 19:17 --- unassigning. The BE workaround bypasses it for me, no time to look further. -- aldot at gcc dot gnu dot org changed: What|Removed |Added

[Bug c/31537] duplicate weakref emitted with IMA

2008-03-14 Thread rguenth at gcc dot gnu dot org
-- rguenth at gcc dot gnu dot org changed: What|Removed |Added Target Milestone|4.3.0 |--- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31537

[Bug c/31537] duplicate weakref emitted with IMA

2008-01-29 Thread aldot at gcc dot gnu dot org
--- Comment #9 from aldot at gcc dot gnu dot org 2008-01-29 21:07 --- Created an attachment (id=15053) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15053&action=view) patch in testing This is a simple fix to adjust the respective vector (that get's filled/finalized far too early,

[Bug c/31537] duplicate weakref emitted with IMA

2008-01-25 Thread aldot at gcc dot gnu dot org
--- Comment #8 from aldot at gcc dot gnu dot org 2008-01-25 18:02 --- Mine. $ cat pr31537.i static int __gthrw_pthread_once __attribute__ ((__weakref__ ("pthread_once"))); $ gcc-4.3-HEAD -S -o - -Os pr31537.i pr31537.i pr31537.i \ pr31537.i pr31537.i pr31537.i -fno-tree-vnhoist -combine

[Bug c/31537] duplicate weakref emitted with IMA

2008-01-25 Thread aldot at gcc dot gnu dot org
--- Comment #7 from aldot at gcc dot gnu dot org 2008-01-25 08:28 --- Testing a patch. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31537

[Bug c/31537] duplicate weakref emitted with IMA

2008-01-02 Thread aldot at gcc dot gnu dot org
--- Comment #6 from aldot at gcc dot gnu dot org 2008-01-02 19:06 --- Any update? Current trunk still produces wrong code for weakrefs.. -- aldot at gcc dot gnu dot org changed: What|Removed |Added --

[Bug c/31537] duplicate weakref emitted with IMA

2007-06-21 Thread aldot at gcc dot gnu dot org
--- Comment #5 from aldot at gcc dot gnu dot org 2007-06-21 16:44 --- Without combine, the attribute is ignored: $ gcc-4.3.orig-HEAD -c pr.c -o /dev/null pr.c: In function 'f1': pr.c:3: warning: '__weakref__' attribute ignored pr.c: In function 'f2': pr.c:7: warning: '__weakref__' att

[Bug c/31537] duplicate weakref emitted with IMA

2007-05-03 Thread geoffk at gcc dot gnu dot org
--- Comment #4 from geoffk at gcc dot gnu dot org 2007-05-03 20:14 --- The following testcase should be equivalent to the original one but not involve IMA: void f1() { static __gthrw_pthread_once __attribute__ ((__weakref__ ("pthread_once"))); } void f2() { static __gthrw_pthread_

[Bug c/31537] duplicate weakref emitted with IMA

2007-05-02 Thread aldot at gcc dot gnu dot org
--- Comment #3 from aldot at gcc dot gnu dot org 2007-05-02 21:43 --- Richi suggested it was a FE bug. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31537

[Bug c/31537] duplicate weakref emitted with IMA

2007-05-02 Thread geoffk at gcc dot gnu dot org
--- Comment #2 from geoffk at gcc dot gnu dot org 2007-05-02 21:23 --- Since these are both 'static', shouldn't they be named things like __gthrw_pthread_once.247 in the assembler? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31537

[Bug c/31537] duplicate weakref emitted with IMA

2007-04-13 Thread aldot at gcc dot gnu dot org
--- Comment #1 from aldot at gcc dot gnu dot org 2007-04-13 14:32 --- Created an attachment (id=13363) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13363&action=view) inaccurate bypass Not a patch. By marking ultimate target's asm_written_flag and bailing if it was already set,