[Bug target/28490] New: [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[Bug target/28490] New: [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

I get the following ICE with gcc 4.2 on ia64 at -O1.  It doesn't happen with
4.0 or 4.1.

tbm@coconut0:~$ /usr/lib/gcc-snapshot/bin/gcc -c -O1 mini.c
mini.c: In function 'readtoken1':
mini.c:60: internal compiler error: in ia64_expand_move, at
config/ia64/ia64.c:1088
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.
For Debian GNU/Linux specific bug reporting instructions,
see <URL:file:///usr/share/doc/gcc-4.1/README.Bugs>.
tbm@coconut0:~$ /usr/lib/gcc-snapshot/bin/gcc -c mini.c
tbm@coconut0:~$ gcc-4.1 -c -O2 mini.c
tbm@coconut0:~$ gcc-4.0 -c -O2 mini.c
tbm@coconut0:~$


-- 
           Summary: [4.2 regression] ICE in ia64_expand_move, at
                    config/ia64/ia64.c:1088
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: tbm at cyrius dot com
GCC target triplet: ia64-linux-gnu


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #1 from tbm at cyrius dot com  2006-07-26 05:41 -------
Created an attachment (id=11944)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11944&action=view)
test case


-- 


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #2 from tbm at cyrius dot com  2006-07-26 05:42 -------
This has started to occur sometime between 20051122 and 20060218.


-- 


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2006-07-26 08:06 -------
Confirmed.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rguenth at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-07-26 08:06:35
               date|                            |


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.2.0


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #4 from tbm at cyrius dot com  2006-07-26 13:26 -------
Created an attachment (id=11945)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11945&action=view)
test case

Testcase from application dcraw


-- 


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #5 from tbm at cyrius dot com  2006-07-26 13:26 -------
Created an attachment (id=11946)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11946&action=view)
test case

Testcase from application dump


-- 


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

[Bug target/28490] [4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #6 from tbm at cyrius dot com  2006-07-26 20:58 -------
Created an attachment (id=11952)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11952&action=view)
test case

Testcase from application "yorick".


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #7 from tbm at cyrius dot com  2006-07-31 11:29 -------
This bug is also present in gcc 4.0 and 4.1 even though my original testcase
didn't expose it there.  My new one does:

tbm@coconut0:~$ gcc-4.0 -c -O ecl-asdf.c
ecl-asdf.c: In function 'LC67traverse':
ecl-asdf.c:81: internal compiler error: in ia64_expand_move, at
config/ia64/ia64.c:1004
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.
For Debian GNU/Linux specific bug reporting instructions,
see <URL:file:///usr/share/doc/gcc-4.0/README.Bugs>.
tbm@coconut0:~$ gcc-4.1 -c -O ecl-asdf.c
ecl-asdf.c: In function 'LC67traverse':
ecl-asdf.c:81: internal compiler error: in ia64_expand_move, at
config/ia64/ia64.c:1052
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.
For Debian GNU/Linux specific bug reporting instructions,
see <URL:file:///usr/share/doc/gcc-4.1/README.Bugs>.
Preprocessed source stored into /tmp/ccQbxgpQ.out file, please attach this to
your bugreport.
tbm@coconut0:~$ /usr/lib/gcc-snapshot/bin/gcc -c -O ecl-asdf.c
ecl-asdf.c: In function 'LC67traverse':
ecl-asdf.c:81: internal compiler error: in ia64_expand_move, at
config/ia64/ia64.c:1088
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.

This is with:

4.0.4 20060630
4.1.2 20060715
4.2.0 20060721


-- 

tbm at cyrius dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|4.2.0                       |4.0.3 4.1.1 4.2.0
      Known to work|4.0.3 4.1.1                 |3.3.6 3.4.6
            Summary|[4.2 regression] ICE in     |[4.0/4.1/4.2 regression] ICE
                   |ia64_expand_move, at        |in ia64_expand_move, at
                   |config/ia64/ia64.c:1088     |config/ia64/ia64.c:1088


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #8 from tbm at cyrius dot com  2006-07-31 11:30 -------
Created an attachment (id=11980)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11980&action=view)
test case for 4.0, 4.1 and 4.2

Testcase from application "ecl".


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[rguenth at gcc dot gnu dot org]

------- Comment #9 from rguenth at gcc dot gnu dot org  2006-07-31 12:20 -------
Reduced testcase from comment #8

typedef union cl_lispunion *cl_object;
union cl_lispunion { cl_object cons; };
typedef union { cl_object data; } cl_symbol_initializer;
extern cl_symbol_initializer cl_symbols[];
extern void cl_funcall (cl_object);
extern int _setjmp(vois);
void LC67traverse (cl_object *lex0)
{
  cl_object T0;
  cl_funcall ((cl_object) (cl_symbols + 1));
  if (_setjmp ())
        ;
  if (lex0[0] == (cl_object) cl_symbols)
        goto L791;
  T0 = ((cl_object) (cl_symbols + 1));
L791:;
  cl_funcall (T0);
}


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[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=28490

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[steven at gcc dot gnu dot org]

------- Comment #10 from steven at gcc dot gnu dot org  2006-08-01 05:51 -------
Why is this a P1 regression?  ia-64 is not a primary platform.


-- 

steven at gcc dot gnu dot org changed:

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


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[mark at codesourcery dot com]

------- Comment #11 from mark at codesourcery dot com  2006-08-02 03:29 -------
Subject: Re:  [4.0/4.1/4.2 regression] ICE in ia64_expand_move,
 at config/ia64/ia64.c:1088

steven at gcc dot gnu dot org wrote:

> Why is this a P1 regression?  ia-64 is not a primary platform.

True -- but IA64 is a secondary platform, and this looks like relatively
straightforward code.  I think it's important.

If we get all the other P1s done, we can argue about whether or not to
ship the compiler. :-)


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[mkuvyrkov at gcc dot gnu dot org]

------- Comment #12 from mkuvyrkov at gcc dot gnu dot org  2006-08-02 13:45 -------
Created an attachment (id=11996)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11996&action=view)
Patch


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[wilson at gcc dot gnu dot org]

------- Comment #13 from wilson at gcc dot gnu dot org  2006-08-04 01:49 -------
*** Bug 28495 has been marked as a duplicate of this bug. ***


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[wilson at gcc dot gnu dot org]

------- Comment #14 from wilson at gcc dot gnu dot org  2006-08-04 01:55 -------
Steve Ellcey posted a patch here:
    http://gcc.gnu.org/ml/gcc-patches/2006-08/msg00016.html
I posted a better patch here:
    http://gcc.gnu.org/ml/gcc-patches/2006-08/msg00054.html
This patch has not been tested yet.

There are also some follow up questions raised in this thread that need to be
answered.


-- 

wilson at gcc dot gnu dot org changed:

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


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[tbm at cyrius dot com]

------- Comment #15 from tbm at cyrius dot com  2006-08-27 18:30 -------
(In reply to comment #14)
> Steve Ellcey posted a patch here:
>     http://gcc.gnu.org/ml/gcc-patches/2006-08/msg00016.html
> I posted a better patch here:
>     http://gcc.gnu.org/ml/gcc-patches/2006-08/msg00054.html
> This patch has not been tested yet.
> 
> There are also some follow up questions raised in this thread that need to be
> answered.

Steve, are you working on clarifying those new questions?



-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[sje at cup dot hp dot com]

------- Comment #16 from sje at cup dot hp dot com  2006-08-28 16:07 -------
Yes, I did some performance measurements with SPEC2000. Allowing any (symbol +
offset) resulted in slightly slower code overall, allowing no (symbol + offset)
resulted in slightly faster code overall.  I will be submitting a patch to do
this but probably not until next week.


-- 


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

[Bug target/28490] [4.0/4.1/4.2 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[wilson at gcc dot gnu dot org]

------- Comment #17 from wilson at gcc dot gnu dot org  2006-09-15 23:05 -------
Subject: Bug 28490

Author: wilson
Date: Fri Sep 15 23:05:40 2006
New Revision: 116983

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=116983
Log:
Short term fix for PR 28490.
* config/ia64/ia64.c (ia64_legitimate_constant_p, cast CONST):
Handle symbol offsets same as they are handled in ia64_expand_move
and move_operand.

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


-- 


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

[Bug target/28490] [4.0/4.1 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[pinskia at gcc dot gnu dot org]

------- Comment #18 from pinskia at gcc dot gnu dot org  2006-09-16 02:46 -------
worked around for 4.2.0 so no longer a regression.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|4.0.3 4.1.1 4.2.0           |4.0.3 4.1.1
            Summary|[4.0/4.1/4.2 regression] ICE|[4.0/4.1 regression] ICE in
                   |in ia64_expand_move, at     |ia64_expand_move, at
                   |config/ia64/ia64.c:1088     |config/ia64/ia64.c:1088
   Target Milestone|4.2.0                       |4.0.4


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

[Bug target/28490] [4.0/4.1 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[sje at gcc dot gnu dot org]

------- Comment #19 from sje at gcc dot gnu dot org  2006-09-19 16:09 -------
Subject: Bug 28490

Author: sje
Date: Tue Sep 19 16:09:48 2006
New Revision: 117057

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117057
Log:
        PR 28490
        * config/ia64/ia64.c (ia64_legitimate_constant_p): Allow function
        pointers as legitimate constants.

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


-- 


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

[Bug target/28490] [4.0/4.1 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[sje at gcc dot gnu dot org]

------- Comment #20 from sje at gcc dot gnu dot org  2006-10-09 18:24 -------
Subject: Bug 28490

Author: sje
Date: Mon Oct  9 18:24:32 2006
New Revision: 117582

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117582
Log:
        PR target/28490
        Backport from mainline
        2006-09-15  Jim Wilson  <wilson@specifix.com>
        2006-09-19  Steve Ellcey  <sje@cup.hp.com>
        * config/ia64/ia64.c (ia64_legitimate_constant_p): Allow function
        pointers as legitimate constants.  Handle symbol offsets same as
        they are handled in ia64_expand_move and move_operand.

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


-- 


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

[Bug target/28490] [4.0/4.1 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[sje at gcc dot gnu dot org]

------- Comment #21 from sje at gcc dot gnu dot org  2006-10-09 18:26 -------
Subject: Bug 28490

Author: sje
Date: Mon Oct  9 18:26:35 2006
New Revision: 117583

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117583
Log:
        PR target/28490
        Backport from mainline
        2006-09-15  Jim Wilson  <wilson@specifix.com>
        2006-09-19  Steve Ellcey  <sje@cup.hp.com>
        * config/ia64/ia64.c (ia64_legitimate_constant_p): Allow function
        pointers as legitimate constants.  Handle symbol offsets same as
        they are handled in ia64_expand_move and move_operand.

Modified:
    branches/gcc-4_0-branch/gcc/ChangeLog
    branches/gcc-4_0-branch/gcc/config/ia64/ia64.c


-- 


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

[Bug target/28490] [4.0/4.1 regression] ICE in ia64_expand_move, at config/ia64/ia64.c:1088

[sje at cup dot hp dot com]

------- Comment #22 from sje at cup dot hp dot com  2006-10-09 18:27 -------
Backported the change to 4.1 and 4.0 branches.  Closing as fixed.


-- 

sje at cup dot hp dot com changed:

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


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