[Bug target/50493] New: ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[Bug target/50493] New: ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[rmansfield at qnx dot com]

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

             Bug #: 50493
           Summary: ICE in neon_disambiguate_copy, at
                    config/arm/arm.c:20388
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: rmansfield@qnx.com
              Host: x86_64-linux-gnu
            Target: arm-unknown-linux-gnueabi
             Build: x86_64-linux-gnu


Created attachment 25347
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25347
preprocessed source

$ ./xgcc -v
Using built-in specs.
COLLECT_GCC=./xgcc
Target: arm-unknown-linux-gnueabi
Configured with: ../configure --target=arm-unknown-linux-gnueabi
--prefix=/home/ryan/x-tools/arm-unknown-linux-gnueabi
--with-sysroot=/home/ryan/x-tools/arm-unknown-linux-gnueabi/arm-unknown-linux-gnueabi//sys-root
--disable-multilib
--with-local-prefix=/home/ryan/x-tools/arm-unknown-linux-gnueabi/arm-unknown-linux-gnueabi/sys-root
--disable-nls --enable-threads=posix --enable-symvers=gnu --enable-c99
--enable-long-long --enable-target-optspace
target_alias=arm-unknown-linux-gnueabi --enable-languages=c++ --disable-shared
--disable-libmudflap --disable-libssp
Thread model: posix
gcc version 4.7.0 20110923 (experimental) [trunk revision 179117] (GCC) 

$ ./xgcc -B. -O3 -mfpu=neon -mfloat-abi=softfp ~/ice.i -fpic
/home/ryan/ice.i: In function 'SetupUSEVertexShader':
/home/ryan/ice.i:342:38: warning: incompatible implicit declaration of built-in
function 'calloc' [enabled by default]
/home/ryan/ice.i:353:1: internal compiler error: in neon_disambiguate_copy, at
config/arm/arm.c:20388
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[mikpe at it dot uu.se]

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

Mikael Pettersson <mikpe at it dot uu.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |irar at gcc dot gnu.org,
                   |                            |mikpe at it dot uu.se

--- Comment #1 from Mikael Pettersson <mikpe at it dot uu.se> 2011-09-24 16:42:17 UTC ---
This test case started ICEing on trunk with Ira's r178588:

http://gcc.gnu.org/ml/gcc-cvs/2011-09/msg00199.html

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[irar at il dot ibm.com]

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

Ira Rosen <irar at il dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011-09-26
                 CC|                            |irar at il dot ibm.com,
                   |                            |ramana at gcc dot gnu.org
     Ever Confirmed|0                           |1

--- Comment #2 from Ira Rosen <irar at il dot ibm.com> 2011-09-26 07:18:29 UTC ---
Looks like the problem is with neon_movv4si. 

My patch enabled the use of quad-word vectors by default, the test doesn't fail
with double-words.

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[michael.hope at linaro dot org]

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

Michael Hope <michael.hope at linaro dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |michael.hope at linaro dot
                   |                            |org

--- Comment #3 from Michael Hope <michael.hope at linaro dot org> 2011-11-09 20:58:41 UTC ---
I'm seeing similar at:
 https://bugs.launchpad.net/gcc/+bug/809446

and:
 https://bugs.launchpad.net/gcc-linaro/+bug/885393

but it's a bit random.

Richard Earnshaw was looking at this recently.  I'll ping him.

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[rearnsha at gcc dot gnu.org]

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

Richard Earnshaw <rearnsha at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2011-09-26 00:00:00         |2011-11-11 0:00
                 CC|                            |rearnsha at gcc dot gnu.org
         AssignedTo|unassigned at gcc dot       |rearnsha at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #4 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-11 15:59:31 UTC ---
Has gone latent again on trunk, but confirmed that the code here is both broken
and unnecessarily complex.  Testing a fix.

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[rearnsha at gcc dot gnu.org]

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

--- Comment #5 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-19 13:47:40 UTC ---
Author: rearnsha
Date: Sat Nov 19 13:47:35 2011
New Revision: 181508

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181508
Log:
PR target/50493
* arm.c (neon_disambiguate_copy): Correctly handle partial overlap
of src and dest operands.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/arm/arm.c

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[rearnsha at gcc dot gnu.org]

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

--- Comment #6 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-19 16:47:06 UTC ---
Author: rearnsha
Date: Sat Nov 19 16:47:02 2011
New Revision: 181510

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181510
Log:
PR target/50493
* arm.c (neon_disambiguate_copy): Correctly handle partial overlap
of src and dest operands.

Modified:
    branches/gcc-4_6-branch/gcc/ChangeLog
    branches/gcc-4_6-branch/gcc/config/arm/arm.c

[Bug target/50493] ICE in neon_disambiguate_copy, at config/arm/arm.c:20388

[rearnsha at gcc dot gnu.org]

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

Richard Earnshaw <rearnsha at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.6.3

--- Comment #7 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-19 16:48:25 UTC ---
Fixed on trunk and 4.6 branch