[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831 Richard Guenther changed: What|Removed |Added Known to work||4.6.3 Target Milestone|---

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread hubicka at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831 Jan Hubicka changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|unassigned at g

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread tetra2005 at gmail dot com
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread markus at trippelsdorf dot de
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831 Markus Trippelsdorf changed: What|Removed |Added CC||markus at trippelsdorf dot

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread markus at trippelsdorf dot de
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.

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831 Richard Guenther changed: What|Removed |Added Attachment #27735|0 |1 is obsolete|

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread rguenth at gcc dot gnu.org
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(?)

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-03 Thread tetra2005 at gmail dot com
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-04 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831 Richard Guenther changed: What|Removed |Added CC||andi-gcc at firstfloor dot

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-04 Thread andi-gcc at firstfloor dot org
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-04 Thread tetra2005 at gmail dot com
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.

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-07-10 Thread tetra2005 at gmail dot com
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:

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-09-20 Thread jakub at gcc dot gnu.org
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-10-06 Thread hubicka at gcc dot gnu.org
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-10-06 Thread hubicka at gcc dot gnu.org
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.

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-10-06 Thread markus at trippelsdorf dot de
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-10-06 Thread hubicka at ucw dot cz
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-10-06 Thread hubicka at gcc dot gnu.org
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

[Bug lto/53831] [4.7/4.8 Regression] Virtuals missing in LTO symtab

2012-12-06 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53831 Richard Biener changed: What|Removed |Added Status|ASSIGNED|WAITING --- Comment #30 from R