[Bug rtl-optimization/30761] New: [4.3 regression] Error: unsupported relocation against sfp

[Bug rtl-optimization/30761] New: [4.3 regression] Error: unsupported relocation against sfp

[schwab at suse dot de]

This is caused by the lower-subreg patch (adding -fno-split-wide-types is a
workaround):

$ ../../xgcc -B../../ -c -O2 -fno-common      -gnatpg -gnata -I- -I../rts -I.
-I ../../../../gcc/ada ../../../../gcc/ada/make.adb -c 
/tmp/ccXcjvZm.s: Assembler messages:
/tmp/ccXcjvZm.s:6958: Error: unsupported relocation against sfp
$ grep sfp make.s
        mr 21,sfp


-- 
           Summary: [4.3 regression] Error: unsupported relocation against
                    sfp
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Keywords: wrong-code, build
          Severity: normal
          Priority: P3
         Component: rtl-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: schwab at suse dot de
GCC target triplet: powerpc-suse-linux


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

[Bug rtl-optimization/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2007-02-11 10:31 -------
Actually this has nothing to do with -fsplit-wide-types, This shows up in some
cases without that, There is a reload patch floating about which fixes this. 
The problem is that reload does not elimate the soft frame pointer.  In fact
this is really a regression in 4.1.0 exposed by:
2005-06-30  Jakub Jelinek  <jakub@redhat.com>

        * config/rs6000/rs6000.h (FIRST_PSEUDO_REGISTER): Increment.
        (DWARF_FRAME_REGISTERS, DWARF_REG_TO_UNWIND_COLUMN): Adjust, so
        that addition of sfp doesn't change these.
        (FIXED_REGISTERS, CALL_USED_REGISTERS, CALL_REALLY_USED_REGISTERS,
 ....


I actually saw it in a modified version of 4.1.1.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 GCC target triplet|powerpc-suse-linux          |powerpc*-*-*
            Summary|[4.3 regression] Error:     |[4.1/4.2/4.3 regression]
                   |unsupported relocation      |Error: unsupported
                   |against sfp                 |relocation against sfp
   Target Milestone|---                         |4.1.2


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

[Bug rtl-optimization/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2007-02-11 10:47 -------
(In reply to comment #1)
> I actually saw it in a modified version of 4.1.1.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |28690
              nThis|                            |
           Keywords|                            |link-failure


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

[Bug middle-end/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2007-02-11 19:12 -------
> This is caused by the lower-subreg patch
Exposed by, not caused by in this case.  It exposed an issue with reload and
register elimination which was originally exposed by Jakub's patch.  I was able
to reproduce the problem on a heavely modified version of 4.1.1 so I know the
bug is there also.
See the patch at:  http://gcc.gnu.org/ml/gcc-patches/2006-12/msg00992.html

Which should fix the problem.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|rtl-optimization            |middle-end
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-02-11 19:12:13
               date|                            |


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

[Bug middle-end/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.2                       |4.1.3


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

[Bug middle-end/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


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

[Bug middle-end/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[uweigand at gcc dot gnu dot org]

------- Comment #4 from uweigand at gcc dot gnu dot org  2007-02-21 15:05 -------
Subject: Bug 30761

Author: uweigand
Date: Wed Feb 21 15:05:01 2007
New Revision: 122199

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=122199
Log:
        PR middle-end/30761
        * reload1.c (eliminate_regs_in_insn): In the single_set special
        case, attempt to re-recognize the insn before falling back to
        having reload fix it up.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/reload1.c


-- 


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

[Bug middle-end/30761] [4.1/4.2/4.3 regression] Error: unsupported relocation against sfp

[mmitchel at gcc dot gnu dot org]

------- Comment #5 from mmitchel at gcc dot gnu dot org  2007-03-12 18:45 -------
Ulrich --

Did your patch fix this PR on mainline?  Is a backport to 4.1/4.2 possible?

Thanks,

-- Mark


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |uweigand at gcc dot gnu dot
                   |                            |org


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

[Bug middle-end/30761] [4.1/4.2 regression] Error: unsupported relocation against sfp

[uweigand at gcc dot gnu dot org]

------- Comment #6 from uweigand at gcc dot gnu dot org  2007-03-12 19:34 -------
I haven't verified that this problem is fixed -- the patch was originally
intended to fix another bug uncovered by Peter Bergner, and I just added
this PR number to the check-in due to Andrew's comment #3 on this bug.

Andrew, have you verified that the problem is fixed now?

I could backport the reload1.c change to 4.1/4.2 -- I haven't done so as
there's always some risk associated with such backports, and it appeared
from this bugzilla record that the bug was exposed only with a 4.3 change
anyway.  Was this impression mistaken?


-- 


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

[Bug middle-end/30761] [4.1/4.2 regression] Error: unsupported relocation against sfp

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2007-03-12 20:19 -------
> Andrew, have you verified that the problem is fixed now?
Yes the patch fixed the problem for me.

>Was this impression mistaken?
Yes but in my own testing of a private tree of GCC 4.1.1 modifed for the PS3
game OS ABI, I found this bug happening there too.


-- 


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

[Bug middle-end/30761] [4.1/4.2 regression] Error: unsupported relocation against sfp

[mmitchel at gcc dot gnu dot org]

------- Comment #8 from mmitchel at gcc dot gnu dot org  2007-04-24 22:46 -------
Ulrich, in response to your question in Comment #6, yes, this bug appears in
4.1 and 4.2, not just in 4.3.  So, if you think it's safe to backport the
reload patch, it would be nice to have the fix there as well.


-- 


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

[Bug middle-end/30761] [4.1/4.2 regression] Error: unsupported relocation against sfp

[uweigand at gcc dot gnu dot org]

------- Comment #9 from uweigand at gcc dot gnu dot org  2007-04-26 22:10 -------
Subject: Bug 30761

Author: uweigand
Date: Thu Apr 26 22:10:09 2007
New Revision: 124199

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=124199
Log:
        PR middle-end/30761
        * reload1.c (eliminate_regs_in_insn): In the single_set special
        case, attempt to re-recognize the insn before falling back to
        having reload fix it up.

Modified:
    branches/gcc-4_2-branch/gcc/ChangeLog
    branches/gcc-4_2-branch/gcc/reload1.c


-- 


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

[Bug middle-end/30761] [4.1/4.2 regression] Error: unsupported relocation against sfp

[uweigand at gcc dot gnu dot org]

------- Comment #10 from uweigand at gcc dot gnu dot org  2007-04-27 14:59 -------
Subject: Bug 30761

Author: uweigand
Date: Fri Apr 27 14:59:21 2007
New Revision: 124219

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=124219
Log:
        PR middle-end/30761
        * reload1.c (eliminate_regs_in_insn): In the single_set special
        case, attempt to re-recognize the insn before falling back to
        having reload fix it up.

Modified:
    branches/gcc-4_1-branch/gcc/ChangeLog
    branches/gcc-4_1-branch/gcc/reload1.c


-- 


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

[Bug middle-end/30761] [4.1/4.2 regression] Error: unsupported relocation against sfp

[uweigand at gcc dot gnu dot org]

------- Comment #11 from uweigand at gcc dot gnu dot org  2007-04-27 15:03 -------
(In reply to comment #8)
> Ulrich, in response to your question in Comment #6, yes, this bug appears in
> 4.1 and 4.2, not just in 4.3.  So, if you think it's safe to backport the
> reload patch, it would be nice to have the fix there as well.

I've back-ported the fix to 4.2 and 4.1 now.


-- 

uweigand at gcc dot gnu dot org changed:

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


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