[Bug c/11587] New: SH ICE in reload_cse_simplify_operand, postreload.c

[Bug c/11587] New: SH ICE in reload_cse_simplify_operand, postreload.c

[eager at mvista dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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

           Summary: SH ICE in reload_cse_simplify_operand, postreload.c
           Product: gcc
           Version: 3.3
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: eager at mvista dot com
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: i686-pc-linux
  GCC host triplet: i686-pc-linux
GCC target triplet: sh4-hardhat-linux

ICE in reload_cse_simplify_operands, at postreload.c:378

tst.c:22: error: insn does not satisfy its constraints:
(insn 90 89 63 7 (set (reg:SI 65 fr1 [orig:158 <result> ] [158])
        (reg:SI 147 t)) 123 {movsi_ie} (insn_list 89 (nil))
    (nil))
tst.c:22: internal compiler error: in reload_cse_simplify_operands, at
postreload.c:378

tst.c:
int foo1(void);
int foo2();
int foo3();

static int goo()
        {
        int i;

        if (i <= 0) return i;

        if (foo1() > 1)
                {
                i = foo2();
                if (i < 0) return i;
                if (i)
                        {
                        if (foo3()) return 1;
                        return 0;
                        }
                }

        }

[Bug optimization/11587] SH ICE in reload_cse_simplify_operand, postreload.c

[pinskia at physics dot uc dot edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at physics dot uc dot edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|c                           |optimization
     Ever Confirmed|                            |1
  GCC build triplet|i686-pc-linux               |*-*-*
   GCC host triplet|i686-pc-linux               |*-*-*
   Last reconfirmed|0000-00-00 00:00:00         |2003-07-19 02:19:07
               date|                            |


------- Additional Comments From pinskia at physics dot uc dot edu  2003-07-19 02:19 -------
Note you forgot to mention what options you used, -O1 was enough to reproduce this.

I can confirm this on 3.3.1 (20030714) and the mainline (20030718).

[Bug optimization/11587] SH ICE in reload_cse_simplify_operand, postreload.c

[dhazeghi at yahoo dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


dhazeghi at yahoo dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  GCC build triplet|*-*-*                       |
   GCC host triplet|*-*-*                       |
   Target Milestone|3.4                         |---

[Bug optimization/11587] SH ICE in reload_cse_simplify_operand, postreload.c

[dank at kegel dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From dank at kegel dot com  2003-09-14 21:16 -------
This did not occur in Hard Hat 2.0's gcc-2.97, nor in dodes' gcc-3.0.2,
so IMHO this is a regression.  I'll submit a patch to add this test to
the testsuite.  Thanks!

[Bug target/11587] [3.3/3.4 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |critical
          Component|optimization                |target
            Summary|SH ICE in                   |[3.3/3.4 Regression] SH ICE
                   |reload_cse_simplify_operand,|in
                   |postreload.c                |reload_cse_simplify_operand,
                   |                            |postreload.c
   Target Milestone|---                         |3.3.2


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-09-14 22:39 -------
The patch seems simple enough for 3.3.2 (note I cannot approve anything).

[Bug target/11587] [3.3/3.4 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[dank at kegel dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From dank at kegel dot com  2003-09-15 15:56 -------
Fix works for me in gcc-3.3.1 as released (haven't tried applying it to branch yet).

[Bug target/11587] [3.3/3.4 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[dank at kegel dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From dank at kegel dot com  2003-09-15 17:44 -------
I've prepared a patch to add a testcase for this PR to the testsuite; see
http://www.kegel.com/crosstool/current/patches/gcc-3.3.1/pr11587-1-test.patch
I'll submit it to gcc-patches in a couple days if nobody squawks.

BTW, the fix for bug 11587 seemed to fix bug 11162 for me.

[Bug target/11587] [3.3/3.4 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[mmitchel at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.3.2                       |3.3.3


------- Additional Comments From mmitchel at gcc dot gnu dot org  2003-10-16 09:23 -------
Postponed until GCC 3.3.3.

[Bug target/11587] [3.3/3.4 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[gdr at gcc dot gnu dot org]

------- Additional Comments From gdr at gcc dot gnu dot org  2003-12-21 20:49 -------
Joern --
Can you comment on the patch provided in the audit trail?
Thanks,

-- Gaby


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |joern dot rennecke at superh
                   |                            |dot com


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

[Bug target/11587] [3.3 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-26 18:25 -------
This no longer ICEs on the mainline.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[3.3/3.4 Regression] SH ICE |[3.3 Regression] SH ICE in
                   |in                          |reload_cse_simplify_operand,
                   |reload_cse_simplify_operand,|postreload.c
                   |postreload.c                |


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

[Bug target/11587] [3.3 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[renneckej at superh dot com]

------- Additional Comments From renneckej at superh dot com  2004-01-06 12:30 -------
Subject: Re:  [3.3 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

The patch tests the wrong condition.  A copy from T to a general purpose
register is unproblematic and cheap.  The need for a secondary reload
arises on the contrary when the other class is not a general purpose register.
See how FPSCR_REGS is handled.



-- 


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

[Bug target/11587] [3.3 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[gdr at gcc dot gnu dot org]

------- Additional Comments From gdr at gcc dot gnu dot org  2004-01-14 08:16 -------
won't fix before 3.3.3

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|critical                    |normal


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

[Bug target/11587] [3.3 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[gdr at gcc dot gnu dot org]

------- Additional Comments From gdr at gcc dot gnu dot org  2004-01-21 04:11 -------
Proposed patch is wrong and the problem is fixed in 3.4.x.
Proper patch for 3.3.x is unlikely.


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


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

[Bug target/11587] [3.3 Regression] SH ICE in reload_cse_simplify_operand, postreload.c

[dank at kegel dot com]

------- Additional Comments From dank at kegel dot com  2004-02-09 16:06 -------
I was surprised and pleased to see that my testcase for this bug
passes if you apply SUGIOKA Toshinobu's backport of the fix for PR13260 to
gcc-3.3.3 (mis-archived as
http://www.m17n.org/linux-sh/ml/linux-sh/2003-02/msg00014.html)!

So those of you using gcc-3.3.3 might want to apply that patch.
In fact, maybe we should consider applying it to gcc-3.3.3 in cvs before release...

-- 


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