[Bug target/52006] New: [4.7 Regression] ARM ICE on

[Bug target/52006] New: [4.7 Regression] ARM ICE on

[jakub at gcc dot gnu.org]

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

             Bug #: 52006
           Summary: [4.7 Regression] ARM ICE on
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: jakub@gcc.gnu.org
        ReportedBy: jakub@gcc.gnu.org
                CC: ramana@gcc.gnu.org
            Target: arm*-linux-gnueabi


/* { dg-do compile } */
/* { dg-options "-march=armv7-a -mfloat-abi=hard -O2 -fPIC" } */

unsigned long a;
static int b;

void
foo (void)
{
  asm volatile ("" : "=r" (b));
}

void
bar (float f)
{
  if (f < b / 100.0)
    a = 1;
}

ICEs on arm with:
error: insn does not satisfy its constraints:
(insn 69 64 10 2 (set (reg:SI 77 s14 [orig:143 b ] [143])
        (mem:SI (unspec:SI [
                    (reg/f:SI 2 r2 [141])
                    (const_int 8 [0x8])
                    (const_int 1 [0x1])
                ] UNSPEC_PIC_BASE) [0 S4 A32])) rh784748.i:17 188
{tls_load_dot_plus_eight}
     (nil))
rh784748.i:19:1: internal compiler error: in copyprop_hardreg_forward_1, at
regcprop.c:767

[Bug target/52006] [4.7 Regression] ARM ICE with incorrectly peepholed tls_load_dot_plus_eight

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-01-26
            Summary|[4.7 Regression] ARM ICE on |[4.7 Regression] ARM ICE
                   |                            |with incorrectly peepholed
                   |                            |tls_load_dot_plus_eight
     Ever Confirmed|0                           |1

[Bug target/52006] [4.7 Regression] ARM ICE with incorrectly peepholed tls_load_dot_plus_eight

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |4.6.3
   Target Milestone|---                         |4.7.0
      Known to fail|                            |4.7.0

[Bug target/52006] [4.7 Regression] ARM ICE with incorrectly peepholed tls_load_dot_plus_eight

[jakub at gcc dot gnu.org]

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

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-01-26 09:35:30 UTC ---
Created attachment 26471
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26471
gcc47-pr52006.patch

Untested fix.

[Bug target/52006] [4.7 Regression] ARM ICE with incorrectly peepholed tls_load_dot_plus_eight

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-01-28 00:33:15 UTC ---
Fixed.

[Bug target/52006] [4.7 Regression] ARM ICE with incorrectly peepholed tls_load_dot_plus_eight

[jakub at gcc dot gnu.org]

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-01-28 00:29:15 UTC ---
Author: jakub
Date: Sat Jan 28 00:29:11 2012
New Revision: 183661

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=183661
Log:
    PR target/52006
    * config/arm/arm.md (pic_add_dot_plus_eight peephole2): Use
    arm_general_register_operand predicate for operand 2 instead of
    register_operand.

    * gcc.target/arm/pr52006.c: New test.

Added:
    trunk/gcc/testsuite/gcc.target/arm/pr52006.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/arm/arm.md
    trunk/gcc/testsuite/ChangeLog

[Bug target/52006] [4.7 Regression] ARM ICE with incorrectly peepholed tls_load_dot_plus_eight

[jakub at gcc dot gnu.org]

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

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-02-09 17:26:01 UTC ---
Author: jakub
Date: Thu Feb  9 17:25:47 2012
New Revision: 184057

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=184057
Log:
    Backported from mainline
    2012-01-28  Jakub Jelinek  <jakub@redhat.com>

    PR target/52006
    * config/arm/arm.md (pic_add_dot_plus_eight peephole2): Use
    arm_general_register_operand predicate for operand 2 instead of
    register_operand.

    * gcc.target/arm/pr52006.c: New test.

Added:
    branches/gcc-4_6-branch/gcc/testsuite/gcc.target/arm/pr52006.c
Modified:
    branches/gcc-4_6-branch/gcc/ChangeLog
    branches/gcc-4_6-branch/gcc/config/arm/arm.md
    branches/gcc-4_6-branch/gcc/testsuite/ChangeLog