[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[wilson at gcc dot gnu dot org]

--- Comment #8 from wilson at gcc dot gnu dot org  2008-01-09 19:11 ---
*** Bug 34266 has been marked as a duplicate of this bug. ***


-- 

wilson at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||kate01123 at gmail dot com


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[sje at gcc dot gnu dot org]

--- Comment #6 from sje at gcc dot gnu dot org  2007-07-02 17:15 ---
Subject: Bug 31684

Author: sje
Date: Mon Jul  2 17:15:35 2007
New Revision: 126216

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126216
Log:
PR target/31684
* haifa-sched.c (add_to_speculative_block): Change copy_rtx to
copy_insn.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/haifa-sched.c


-- 


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[sje at cup dot hp dot com]

--- Comment #7 from sje at cup dot hp dot com  2007-07-02 17:16 ---
Patch applied.


-- 

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=31684

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[mmitchel at gcc dot gnu dot org]

--- Comment #5 from mmitchel at gcc dot gnu dot org  2007-06-29 18:14 
---
This patch is OK.  Steve, please commit.


-- 

mmitchel at gcc dot gnu dot org changed:

   What|Removed |Added

   Priority|P3  |P1


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[tbm at cyrius dot com]

--- Comment #4 from tbm at cyrius dot com  2007-06-19 06:39 ---
(In reply to comment #3)
 I tested the patch on IA64 HP-UX and Linux and verified that it fixed the bug
 and caused no regressions.  Jim, do you want to check this patch in?

Given that Jim hasn't answered yet, maybe you can commit the patch yourself.


-- 


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[sje at cup dot hp dot com]

--- Comment #3 from sje at cup dot hp dot com  2007-05-09 23:12 ---
I tested the patch on IA64 HP-UX and Linux and verified that it fixed the bug
and caused no regressions.  Jim, do you want to check this patch in?


-- 


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||pinskia at gcc dot gnu dot
   ||org
   Target Milestone|--- |4.3.0


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[wilson at gcc dot gnu dot org]

--- Comment #1 from wilson at gcc dot gnu dot org  2007-05-01 01:53 ---
It dies in recog because of an invalid extended asm insn rtl.  In an asm with 2
or more outputs, we end up with 2 or more copies of the input vector.  These
vectors must be shared.  They are not in this case.

The problem occurs in haifa-sched, in the speculative instruction support.  It
uses copy_rtx to make a copy of an insn when speculatively scheduling it. 
However, copy_rtx does not preserve the special sharing semantics of asm
operands, so the result is a corrupted insn.  There exists a special routine
copy_insn to solve this problem.  So the solution here is to just replace the
copy_rtx call with a copy_insn call.  This works on mainline for this testcase
but is not otherwise tested.


-- 

wilson at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||wilson at gcc dot gnu dot
   ||org
 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2007-05-01 01:53:27
   date||


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

[Bug target/31684] [4.3 Regression] ICE in get_attr_first_insn, at config/ia64/itanium2.md:1839 at -O2

[wilson at gcc dot gnu dot org]

--- Comment #2 from wilson at gcc dot gnu dot org  2007-05-01 01:56 ---
Created an attachment (id=13466)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13466action=view)
Replace copy_rtx call with copy_insn call.

This works on mainline for the testcase, but has not otherwise been tested.


-- 


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