[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[dhazeghi@yahoo.com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


dhazeghi@yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|dhazeghi@yahoo.com          |
   Last reconfirmed|0000-00-00 00:00:00         |2003-06-05 03:17:44
               date|                            |


------- Additional Comments From dhazeghi@yahoo.com  2003-06-05 03:17 -------
Confirmed with gcc 3.3 branch and mainline (20030604).



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[pinskia@physics.uc.edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia@physics.uc.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |3.3.1

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[pinskia@physics.uc.edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From pinskia@physics.uc.edu  2003-06-11 23:07 -------
This has been downgraded once for 3.2 and then ignored for 3.3, will this bug get fixed 
or will it be ignored for another major release?

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[pinskia at physics dot uc dot edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From pinskia at physics dot uc dot edu  2003-07-07 13:30 -------
Looks like this was fixed at one point and now is broken again:
>From Phil's regression hunter:
: Search converges between 2003-06-04-trunk (#211) and 2003-06-05-trunk (#212)

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[mmitchel at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.3.1                       |3.3.2


------- Additional Comments From mmitchel at gcc dot gnu dot org  2003-07-15 17:16 -------
This bug has a long history!

The solution where only one commutative marker is obeyed makes sense, but it's
not worth holding up 3.3.1 for that.  So this bug is once again postponed --
until GCC 3.3.2.

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[mmitchel at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.3.2                       |3.4


------- Additional Comments From mmitchel at gcc dot gnu dot org  2003-10-16 02:18 -------
And postponed again -- until GCC 3.4.

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-11-30 09:04 -------
The date I quoted is the date it was semi fixed as the tree-ssa produced the same ICE 
untill 2003-11-14.  So most likely the bug is still there but the testcase no longer 
produces an ICE :) (Pushing it to 3.5 as the removal of reload did not make the cut for 3.4 
and it no longer ICE's).

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4                         |3.5


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

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[giovannibajo at libero dot it]

------- Additional Comments From giovannibajo at libero dot it  2004-01-14 04:06 -------
The code is indeed invalid now. Moving back the milestone to 3.4.0 since it's 
still an ICE.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.5.0                       |3.4.0


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

[Bug inline-asm/6162] [3.3/3.4/3.5 regression] i386 asm reloader ice in reload_cse_simplify_operands

[ian at wasabisystems dot com]

------- Additional Comments From ian at wasabisystems dot com  2004-01-29 03:03 -------
Here is a patch:

http://gcc.gnu.org/ml/gcc-patches/2004-01/msg03214.html

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch


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

[Bug inline-asm/6162] [3.3/3.4/3.5 regression] i386 asm reloader ice in reload_cse_simplify_operands

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-29 18:40 -------
Subject: Bug 6162

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	ian@gcc.gnu.org	2004-01-29 18:40:02

Modified files:
	gcc            : ChangeLog reload.c 

Log message:
	PR inline-asm/6162
	* reload.c (find_reloads): Only support one pair of commutative
	operands.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.2553&r2=2.2554
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/reload.c.diff?cvsroot=gcc&r1=1.230&r2=1.231



-- 


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

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[ian at wasabisystems dot com]

------- Additional Comments From ian at wasabisystems dot com  2004-01-29 18:41 -------
Changed summary--I checked in a patch so this is no longer a 3.5 regression.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[3.3/3.4/3.5 regression]    |[3.3/3.4 regression] i386
                   |i386 asm reloader ice in    |asm reloader ice in
                   |reload_cse_simplify_operands|reload_cse_simplify_operands


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

[Bug inline-asm/6162] [3.3/3.4 regression] i386 asm reloader ice in reload_cse_simplify_operands

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-02-12 16:07 -------
Subject: Bug 6162

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	ian@gcc.gnu.org	2004-02-12 16:07:44

Modified files:
	gcc            : ChangeLog reload.c 

Log message:
	PR inline-asm/6162
	* reload.c (find_reloads): Only support one pair of commutative
	operands.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.185&r2=2.2326.2.186
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/reload.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.229.4.3&r2=1.229.4.4



-- 


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

[Bug inline-asm/6162] [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

[ian at wasabisystems dot com]

------- Additional Comments From ian at wasabisystems dot com  2004-02-12 17:01 -------
Fixed for 3.4--changed summary to indicate only 3.3 regression, changed target
milestone from 3.4.0 to 3.3.4.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[3.3/3.4 regression] i386   |[3.3 regression] i386 asm
                   |asm reloader ice in         |reloader ice in
                   |reload_cse_simplify_operands|reload_cse_simplify_operands
   Target Milestone|3.4.0                       |3.3.4


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

[Bug inline-asm/6162] [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

[gdr at gcc dot gnu dot org]

------- Additional Comments From gdr at gcc dot gnu dot org  2004-03-05 12:04 -------
(In reply to comment #28)
> Changed summary--I checked in a patch so this is no longer a 3.5 regression.

Ian --

You fixed this too in gcc-3.5.x and gcc-3.4.x. I would think the patch would
be a good candidate for gcc-3.3.4.  Do you concur?

-- gaby


-- 


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

[Bug inline-asm/6162] [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

[ian at wasabisystems dot com]

------- Additional Comments From ian at wasabisystems dot com  2004-03-05 12:52 -------
Subject: Re:  [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

"gdr at gcc dot gnu dot org" <gcc-bugzilla@gcc.gnu.org> writes:

> You fixed this too in gcc-3.5.x and gcc-3.4.x. I would think the patch would
> be a good candidate for gcc-3.3.4.  Do you concur?

Yes.  This could be done in a particularly safe manner by not omitting
the 
	      else if (!this_insn_is_asm)
		abort ();

That is just a sanity check on the gcc sources, and it did trip one
case in the MIPS backend.  It is not required for correct operation.

I'll submit this too.

Ian


-- 


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

[Bug inline-asm/6162] [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-03-05 17:09 -------
Subject: Bug 6162

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_3-branch
Changes by:	ian@gcc.gnu.org	2004-03-05 17:09:47

Modified files:
	gcc            : ChangeLog reload.c 

Log message:
	PR inline-asm/6162
	* reload.c (find_reloads): Only support one pair of commutative
	operands.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_3-branch&r1=1.16114.2.918&r2=1.16114.2.919
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/reload.c.diff?cvsroot=gcc&only_with_tag=gcc-3_3-branch&r1=1.199.2.12&r2=1.199.2.13



-- 


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

[Bug inline-asm/6162] [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

[ian at wasabisystems dot com]

------- Additional Comments From ian at wasabisystems dot com  2004-03-05 17:14 -------
Bug is now fixed on 3.3 branch, on 3.4 branch, and on mainline.

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


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

[Bug inline-asm/6162] [3.3 regression] i386 asm reloader ice in reload_cse_simplify_operands

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-03-08 10:17 -------
Subject: Bug 6162

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	hammer-3_3-branch
Changes by:	matz@gcc.gnu.org	2004-03-08 10:17:40

Modified files:
	gcc            : ChangeLog.hammer flow.c reload.c 

Log message:
	2004-03-05  Ian Lance Taylor  <ian@wasabisystems.com>
	
	PR inline-asm/6162
	* reload.c (find_reloads): Only support one pair of commutative
	operands.
	
	2004-02-27  Ian Lance Taylor  <ian@wasabisystems.com>
	
	PR optimization/7871
	* flow.c (mark_set_1): Don't add LOG_LINKS for global registers
	from or to call insns.
	
	2004-02-27  Eric Botcazou  <ebotcazou@libertysurf.fr>
	
	PR optimization/7871
	* flow.c (propagate_one_insn): Interpret calls as setting global
	registers, not merely clobbering them.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.hammer.diff?cvsroot=gcc&only_with_tag=hammer-3_3-branch&r1=1.1.2.368&r2=1.1.2.369
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/flow.c.diff?cvsroot=gcc&only_with_tag=hammer-3_3-branch&r1=1.540.2.9&r2=1.540.2.10
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/reload.c.diff?cvsroot=gcc&only_with_tag=hammer-3_3-branch&r1=1.197.2.13&r2=1.197.2.14



-- 


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