[Bug regression/42113] New: [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[Bug regression/42113] New: [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[mattst88 at gmail dot com]

Unfortunately, but 8603 seems to cause internal compiler errors on select files
when using -O3. -O{s,0,1,2} are unaffected.

I'm attaching pp.i (preprocessed pp.c from libperl), and flist.i (preprocessed
flist.c from rsync) as test cases.

gcc-4.3.4 does not exhibit this failure, but when patched from bug 8603, it
fails. gcc-4.4.1 does not exhibit this failure, but gcc-4.4.2, which includes
the patch from 8603, does fail.


-- 
           Summary: [4.3/4.4/4.5 Regression] Internal Compiler error with -
                    O3, breaking commit known
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: major
          Priority: P3
         Component: regression
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: mattst88 at gmail dot com
 GCC build triplet: alpha-unknown-linux-gnu
  GCC host triplet: alpha-unknown-linux-gnu
GCC target triplet: alpha-unknown-linux-gnu


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

[Bug regression/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[mattst88 at gmail dot com]

------- Comment #1 from mattst88 at gmail dot com  2009-11-20 00:44 -------
Created an attachment (id=19061)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19061&action=view)
Test Case 1 - pp.i - preprocessed pp.c from libperl


-- 


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

[Bug regression/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[mattst88 at gmail dot com]

------- Comment #2 from mattst88 at gmail dot com  2009-11-20 00:45 -------
Created an attachment (id=19062)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19062&action=view)
Test Case 2 - flist.i - preprocessed flist.c from rsync


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #3 from ubizjak at gmail dot com  2009-11-20 10:06 -------
Both files compile OK with 4.5 cross from x86_64-pc-linux-gnu.

Probably a target specific patch should be backported from mainline to 4.3 and
4.4 branches.


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #4 from ubizjak at gmail dot com  2009-11-20 10:42 -------
Confirmed with 4.4 cross.


-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|ubizjak at gmail dot com    |
         AssignedTo|unassigned at gcc dot gnu   |ubizjak at gmail dot com
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-11-20 10:42:41
               date|                            |


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

[Bug target/42113] [4.3/4.4 Regression] Internal Compiler error with -O3, breaking commit known

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4
            Summary|[4.3/4.4/4.5 Regression]    |[4.3/4.4 Regression]
                   |Internal Compiler error with|Internal Compiler error with
                   |-O3, breaking commit known  |-O3, breaking commit known
   Target Milestone|---                         |4.4.3


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

[Bug target/42113] [4.3/4.4 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #5 from ubizjak at gmail dot com  2009-11-20 12:58 -------
This reduced testcase fails also on 4.5:

--cut here--
int make_file (int a, int b)
{
  int foo = a * sizeof (int);

  if (b)
    foo += sizeof (int);

  return foo;
}
--cut here--


-- 


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

[Bug target/42113] [4.3/4.4 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #6 from ubizjak at gmail dot com  2009-11-20 13:40 -------
Created an attachment (id=19067)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=19067&action=view)
patch to fix mode of scratch register in *cmp_s{add,sub}_si{,di}

This patch fixes wrong modes of scratch register in *cmp_s{add,sub}_si{,di}
patterns.


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

-- 

ubizjak at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2009-
                   |                            |11/msg01115.html
   Target Milestone|4.4.3                       |4.3.5


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[mattst88 at gmail dot com]

------- Comment #7 from mattst88 at gmail dot com  2009-11-21 16:15 -------
I can confirm that the attached patch fixes the issue. Thanks!


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[uros at gcc dot gnu dot org]

------- Comment #8 from uros at gcc dot gnu dot org  2009-11-22 09:31 -------
Subject: Bug 42113

Author: uros
Date: Sun Nov 22 09:30:50 2009
New Revision: 154423

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154423
Log:
        PR target/42113
        * config/alpha/alpha.md (*cmp_sadd_si): Change mode
        of scratch register to SImode.
        (*cmp_sadd_sidi): Ditto.
        (*cmp_ssub_si): Ditto.
        (*cmp_ssub_sidi): Ditto.

testsuite/ChangeLog:

        PR target/42113
        * gcc.target/alpha/pr42113.c: New test.


Added:
    trunk/gcc/testsuite/gcc.target/alpha/pr42113.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/alpha/alpha.md
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[mattst88 at gmail dot com]

------- Comment #9 from mattst88 at gmail dot com  2009-11-22 17:52 -------
WRT the test suite: should it be 

/* { dg-options "-O2" } */

or

/* { dg-options "-O3" } */

That is, -O2 or -O3? I could only produce the internal compiler error with -O3,
and not at all with -Os, -O0, -O1, -O2.


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #10 from ubizjak at gmail dot com  2009-11-22 18:10 -------
Subject: Re:  [4.3/4.4/4.5 Regression] Internal Compiler
 error with -O3, breaking commit known

On 11/22/2009 06:52 PM, mattst88 at gmail dot com wrote:
> ------- Comment #9 from mattst88 at gmail dot com  2009-11-22 17:52 -------
> WRT the test suite: should it be
>
> /* { dg-options "-O2" } */
>
> or
>
> /* { dg-options "-O3" } */
>
> That is, -O2 or -O3? I could only produce the internal compiler error with -O3,
> and not at all with -Os, -O0, -O1, -O2.
>    

The testcase also fails with -O2,.

Uros.


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[uros at gcc dot gnu dot org]

------- Comment #11 from uros at gcc dot gnu dot org  2009-11-23 21:14 -------
Subject: Bug 42113

Author: uros
Date: Mon Nov 23 21:14:32 2009
New Revision: 154464

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154464
Log:
        PR target/42113
        * config/alpha/alpha.md (*cmp_sadd_si): Change mode
        of scratch register to SImode.
        (*cmp_sadd_sidi): Ditto.
        (*cmp_ssub_si): Ditto.
        (*cmp_ssub_sidi): Ditto.

testsuite/ChangeLog:

        PR target/42113
        * gcc.target/alpha/pr42113.c: New test.


Added:
    branches/gcc-4_4-branch/gcc/testsuite/gcc.target/alpha/pr42113.c
Modified:
    branches/gcc-4_4-branch/gcc/ChangeLog
    branches/gcc-4_4-branch/gcc/config/alpha/alpha.md
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[uros at gcc dot gnu dot org]

------- Comment #12 from uros at gcc dot gnu dot org  2009-11-23 21:27 -------
Subject: Bug 42113

Author: uros
Date: Mon Nov 23 21:27:30 2009
New Revision: 154465

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=154465
Log:
        PR target/42113
        * config/alpha/alpha.md (*cmp_sadd_si): Change mode
        of scratch register to SImode.
        (*cmp_sadd_sidi): Ditto.
        (*cmp_ssub_si): Ditto.
        (*cmp_ssub_sidi): Ditto.

testsuite/ChangeLog:

        PR target/42113
        * gcc.target/alpha/pr42113.c: New test.


Added:
    branches/gcc-4_3-branch/gcc/testsuite/gcc.target/alpha/pr42113.c
Modified:
    branches/gcc-4_3-branch/gcc/ChangeLog
    branches/gcc-4_3-branch/gcc/config/alpha/alpha.md
    branches/gcc-4_3-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #13 from ubizjak at gmail dot com  2009-11-23 21:30 -------
Fixed.


-- 

ubizjak at gmail dot com changed:

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


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[uros at gcc dot gnu dot org]

------- Comment #14 from uros at gcc dot gnu dot org  2010-03-26 18:46 -------
Subject: Bug 42113

Author: uros
Date: Fri Mar 26 18:45:56 2010
New Revision: 157759

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157759
Log:
        PR target/42113
        * config/alpha/alpha.md (*cmp_sadd_si): Change mode
        of scratch register to DImode.  Split to DImode comparison operator.
        Use SImode subreg of scratch register in the multiplication.
        (*cmp_sadd_sidi): Ditto.
        (*cmp_ssub_si): Ditto.
        (*cmp_ssub_sidi): Ditto.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/alpha/alpha.md


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[uros at gcc dot gnu dot org]

------- Comment #15 from uros at gcc dot gnu dot org  2010-03-27 12:09 -------
Subject: Bug 42113

Author: uros
Date: Sat Mar 27 12:09:24 2010
New Revision: 157774

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157774
Log:
        PR target/42113
        * config/alpha/alpha.md (*cmp_sadd_si): Change mode
        of scratch register to DImode.  Split to DImode comparison operator.
        Use SImode subreg of scratch register in the multiplication.
        (*cmp_sadd_sidi): Ditto.
        (*cmp_ssub_si): Ditto.
        (*cmp_ssub_sidi): Ditto.


Modified:
    branches/gcc-4_4-branch/gcc/ChangeLog
    branches/gcc-4_4-branch/gcc/config/alpha/alpha.md


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[uros at gcc dot gnu dot org]

------- Comment #16 from uros at gcc dot gnu dot org  2010-03-27 12:16 -------
Subject: Bug 42113

Author: uros
Date: Sat Mar 27 12:15:50 2010
New Revision: 157775

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157775
Log:
        PR target/42113
        * config/alpha/alpha.md (*cmp_sadd_si): Change mode
        of scratch register to DImode.  Split to DImode comparison operator.
        Use SImode subreg of scratch register in the multiplication.
        (*cmp_sadd_sidi): Ditto.
        (*cmp_ssub_si): Ditto.
        (*cmp_ssub_sidi): Ditto.


Modified:
    branches/gcc-4_3-branch/gcc/ChangeLog
    branches/gcc-4_3-branch/gcc/config/alpha/alpha.md


-- 


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

[Bug target/42113] [4.3/4.4/4.5 Regression] Internal Compiler error with -O3, breaking commit known

[ubizjak at gmail dot com]

------- Comment #17 from ubizjak at gmail dot com  2010-03-27 12:17 -------
Fixed in a better way.


-- 


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