http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Richard Biener changed:
What|Removed |Added
Status|ASSIGNED|WAITING
--- Comment #30 from R
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #29 from Jan Hubicka 2012-10-06
17:30:51 UTC ---
Author: hubicka
Date: Sat Oct 6 17:30:42 2012
New Revision: 192166
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192166
Log:
PR lto/53831
PR lto/54776
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #28 from Jan Hubicka 2012-10-06 16:20:57
UTC ---
> There is also H.J.'s proposal to add automatic plugin support, or
> mine to add an environment variable. See:
> http://sourceware.org/ml/binutils/2012-10/msg00050.html
Interes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #27 from Markus Trippelsdorf
2012-10-06 16:06:03 UTC ---
(In reply to comment #26)
> Also note that binutils has default search path for plugin. If we installed
> our
> linker plugin there, the ugly gcc-nm/gcc-ar wrappers woul
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #26 from Jan Hubicka 2012-10-06
15:30:12 UTC ---
Also note that binutils has default search path for plugin. If we installed our
linker plugin there, the ugly gcc-nm/gcc-ar wrappers would not be needed.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #25 from Jan Hubicka 2012-10-06
14:50:27 UTC ---
What you hit here is the V1 linker plugin API hack. We, for purpose, hide
COMDAT objects when we know we can hide them, because otherwise linker will
assign them PREVAILING and
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Jakub Jelinek changed:
What|Removed |Added
Target Milestone|4.7.2 |4.7.3
--- Comment #24 from Jaku
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #23 from Yuri Gribov 2012-07-10
15:34:02 UTC ---
The C++ Standard says that "an inline function shall be defined in every
translation unit in which it is used" (n1905, 7.1.2). The test in question
violates this rule: definition for C:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #22 from Yuri Gribov 2012-07-04
12:32:08 UTC ---
> For non fat ("slim") LTO builds you need to use these tools yes
So it seems that original testcase with fat files which used plain ar is indeed
correct and we have a real bug.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #21 from Andi Kleen 2012-07-04
12:24:45 UTC ---
For non fat ("slim") LTO builds you need to use these tools yes. ar needs to
know the symbol table for its index and that needs LTO specific knowledge.
This follows other slim lto compi
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Richard Guenther changed:
What|Removed |Added
CC||andi-gcc at firstfloor dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #19 from Yuri Gribov 2012-07-03
16:55:27 UTC ---
(In reply to comment #18)
> The testcase is fixed when using -fno-fat-lto-objects and gcc-ar
For me x64_64-linux version works if I use gcc-ar (both with and without
-fno-fat-lto-obje
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #18 from Richard Guenther 2012-07-03
14:13:38 UTC ---
That leaves us with the testcase that eventually is invalid. The testcase
is fixed when using -fno-fat-lto-objects and gcc-ar (which is not required,
or should not be required(?)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Richard Guenther changed:
What|Removed |Added
Attachment #27735|0 |1
is obsolete|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #16 from Markus Trippelsdorf
2012-07-03 13:46:43 UTC ---
IOW with:
% cat =ar
#!/bin/sh
cmd=$1
shift
/usr/bin/ar $cmd --plugin
/usr/libexec/gcc/x86_64-pc-linux-gnu/4.8.0/liblto_plugin.so $*
I cannot reproduce the issue anymore.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Markus Trippelsdorf changed:
What|Removed |Added
CC||markus at trippelsdorf dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
--- Comment #14 from Yuri Gribov 2012-07-03
13:31:28 UTC ---
> you need to use the LTO aware nm that gcc installs, gcc-nm. It says
My bad.
> Btw, removing the 'inline' keyword everywhere still reproduces the issue.
> That makes the testcase ce
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Jan Hubicka changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|unassigned at g
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831
Richard Guenther changed:
What|Removed |Added
Known to work||4.6.3
Target Milestone|---
19 matches
Mail list logo