[Bug target/17828] New: -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[Bug target/17828] New: -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at gcc dot gnu dot org]

--- gcc/testsuite/g++.old-deja/g++.other/comdat5.C.jj   2004-10-04
11:20:08.335577980 +0200
+++ gcc/testsuite/g++.old-deja/g++.other/comdat5.C      2004-10-04
11:19:49.362989353 +0200
@@ -0,0 +1,5 @@
+// { dg-do link }
+// { dg-additional-sources " comdat5-aux.cc" }
+// { dg-options "-O2 -fPIC" }
+
+#include "comdat4.C"
--- gcc/testsuite/g++.old-deja/g++.other/comdat5-aux.cc.jj      2004-10-04
11:20:03.496448065 +0200
+++ gcc/testsuite/g++.old-deja/g++.other/comdat5-aux.cc 2004-10-04
11:19:59.249211735 +0200
@@ -0,0 +1,1 @@
+#include "comdat4-aux.cc"

fails on powerpc-redhat-linux.
The local label at the start of jump table is spilled into .got2 section.

-- 
           Summary: -O2 -fPIC doesn't work with switches in linkonce
                    functions and new binutils
           Product: gcc
           Version: 3.4.3
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jakub at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org
GCC target triplet: powerpc-redhat-linux


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at gcc dot gnu dot org]

------- Additional Comments From jakub at gcc dot gnu dot org  2004-10-04 09:44 -------
GCC 3.3.x has the same problem.
Not sure if we want to backport COMDAT patches to GCC 3.3 and 3.4.
Other possible alternative would be to:
a) change ppc32 linker script, so that .got2 section includes both .got2
   and .gnu.linkonce.got2.* sections
b) change rs6000 backend, so that it uses per-function .got2 sections
   for linkonce functions
Or change ld to not error on this problem for another 3 years until all compilers
use COMDAT.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |amodra at bigpond dot net
                   |                            |dot au


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at gcc dot gnu dot org]

------- Additional Comments From jakub at gcc dot gnu dot org  2004-10-04 09:48 -------
Well, the COMDAT changes on HEAD don't solve this problem either.
So I guess we need either per-function .got2 for comdat functions, 
or revert the binutils warnings (or at least turn them off by default).

-- 


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[dwmw2 at infradead dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dwmw2 at infradead dot org


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[amodra at bigpond dot net dot au]

------- Additional Comments From amodra at bigpond dot net dot au  2004-10-05 11:32 -------
If we turn off this ld warning, then we lose sight of the fact that gcc does
have a bug here.  gcc ought to be using linkonce for all sections emitted for a
given function if any section is linkonce (waiver granted for debug and other
sections that the linker edits).

-- 


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-10-05 11:47 -------
Confirmed.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2004-10-05 11:47:43
               date|                            |


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[amodra at bigpond dot net dot au]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |amodra at bigpond dot net
                   |dot org                     |dot au
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2004-10-05 11:47:43         |2005-05-19 11:24:54
               date|                            |


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[olh at suse dot de]

------- Additional Comments From olh at suse dot de  2005-07-25 11:53 -------
Is there a workaround (some --foo option for gcc/ld) for these link errors?


-- 


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[amodra at bigpond dot net dot au]

------- Additional Comments From amodra at bigpond dot net dot au  2005-07-27 02:23 -------
http://gcc.gnu.org/ml/gcc-patches/2005-07/msg01757.html

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[amodra at bigpond dot net dot au]

------- Additional Comments From amodra at bigpond dot net dot au  2005-07-29 02:50 -------
Fixed by linker patch
http://sources.redhat.com/ml/binutils-cvs/2005-07/msg00201.html

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at redhat dot com]

------- Additional Comments From jakub at redhat dot com  2005-07-29 06:54 -------
Do we want to check the comdat5* testcase in?

-- 


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at gcc dot gnu dot org]

------- Comment #12 from jakub at gcc dot gnu dot org  2005-12-08 21:54 -------
Subject: Bug 17828

Author: jakub
Date: Thu Dec  8 21:53:59 2005
New Revision: 108251

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=108251
Log:
        PR target/17828
        * g++.old-deja/g++.other/comdat5.C: New test.
        * g++.old-deja/g++.other/comdat5-aux.cc: New file.

Added:
    branches/gcc-4_1-branch/gcc/testsuite/g++.old-deja/g++.other/comdat5-aux.cc
    branches/gcc-4_1-branch/gcc/testsuite/g++.old-deja/g++.other/comdat5.C
Modified:
    branches/gcc-4_1-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at gcc dot gnu dot org]

------- Comment #11 from jakub at gcc dot gnu dot org  2005-12-08 21:47 -------
Subject: Bug 17828

Author: jakub
Date: Thu Dec  8 21:47:10 2005
New Revision: 108245

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=108245
Log:
        PR target/17828
        * g++.old-deja/g++.other/comdat5.C: New test.
        * g++.old-deja/g++.other/comdat5-aux.cc: New file.

Added:
    trunk/gcc/testsuite/g++.old-deja/g++.other/comdat5-aux.cc
    trunk/gcc/testsuite/g++.old-deja/g++.other/comdat5.C
Modified:
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug target/17828] -O2 -fPIC doesn't work with switches in linkonce functions and new binutils

[jakub at gcc dot gnu dot org]

------- Comment #10 from jakub at gcc dot gnu dot org  2005-11-21 09:52 -------
Subject: Bug 17828

Author: jakub
Date: Mon Nov 21 09:52:20 2005
New Revision: 107297

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=107297
Log:
        PR target/17828
        * g++.old-deja/g++.other/comdat5.C: New test.
        * g++.old-deja/g++.other/comdat5-aux.cc: New file.

Added:
   
branches/gcc-4_0-rhl-branch/gcc/testsuite/g++.old-deja/g++.other/comdat5-aux.cc
    branches/gcc-4_0-rhl-branch/gcc/testsuite/g++.old-deja/g++.other/comdat5.C
Modified:
    branches/gcc-4_0-rhl-branch/gcc/testsuite/ChangeLog


-- 


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