Re: optimization/6845: ICE with -O -march=pentium3/pentium2/athlon

Re: optimization/6845: ICE with -O -march=pentium3/pentium2/athlon

[rth]

Synopsis: ICE with -O -march=pentium3/pentium2/athlon

Responsible-Changed-From-To: unassigned->rth
Responsible-Changed-By: rth
Responsible-Changed-When: Wed Oct  2 13:39:43 2002
Responsible-Changed-Why:
    .
State-Changed-From-To: open->closed
State-Changed-By: rth
State-Changed-When: Wed Oct  2 13:39:43 2002
State-Changed-Why:
    http://gcc.gnu.org/ml/gcc-patches/2002-10/msg00121.html

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=6845

Re: optimization/6845: ICE with -O -march=pentium3/pentium2/athlon

[Torbjörn Wassberg]

The following reply was made to PR optimization/6845; it has been noted by GNATS.

From: =?iso-8859-1?q?Torbj=F6rn=20Wassberg?= <torbjorn.wassberg@swipnet.se>
To: rth@gcc.gnu.org, gcc-bugs@gcc.gnu.org, gcc-prs@gcc.gnu.org,
	nobody@gcc.gnu.org, torbjorn.wassberg@swipnet.se,
	gcc-gnats@gcc.gnu.org
Cc:  
Subject: Re: optimization/6845: ICE with -O -march=pentium3/pentium2/athlon
Date: Tue, 28 May 2002 22:56:52 +0200

 On Tuesday 28 May 2002 20.12, you wrote:
 > Synopsis: ICE with -O -march=pentium3/pentium2/athlon
 >
 > State-Changed-From-To: open->feedback
 > State-Changed-By: rth
 > State-Changed-When: Tue May 28 11:12:39 2002
 > State-Changed-Why:
 >     Should have been fixed by
 >
 >     Sun May 26 14:00:44 CEST 2002  Jan Hubicka  <jh@suse.cz>
 >
 >             * reload.c (find_valid_class): Accept new argument DEST,
 >             choose class accordingly.
 >             (push_reload): Update callers.
 >
 >     Can you confirm?
 >
 > http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&
 >pr=6845
 still ICE in gcc version 3.2 20020528 (experimental), reload.c was already 
 updated the first time

Re: optimization/6845: ICE with -O -march=pentium3/pentium2/athlon

[rth]

Synopsis: ICE with -O -march=pentium3/pentium2/athlon

State-Changed-From-To: feedback->open
State-Changed-By: rth
State-Changed-When: Tue May 28 14:02:38 2002
State-Changed-Why:
    Not fixed.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=6845

Re: optimization/6845: ICE with -O -march=pentium3/pentium2/athlon

[rth]

Synopsis: ICE with -O -march=pentium3/pentium2/athlon

State-Changed-From-To: open->feedback
State-Changed-By: rth
State-Changed-When: Tue May 28 11:12:39 2002
State-Changed-Why:
    Should have been fixed by 
    
    Sun May 26 14:00:44 CEST 2002  Jan Hubicka  <jh@suse.cz>
    
            * reload.c (find_valid_class): Accept new argument DEST,
            choose class accordingly.
            (push_reload): Update callers.
    
    Can you confirm?

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=6845

optimization/6845: ICE with -O -march=pentium3/pentium2/athlon

[torbjorn.wassberg]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1809 bytes --]


>Number:         6845
>Category:       optimization
>Synopsis:       ICE with -O -march=pentium3/pentium2/athlon
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          ice-on-legal-code
>Submitter-Id:   net
>Arrival-Date:   Tue May 28 06:56:00 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator:     Torbjörn Wassberg
>Release:        gcc version 3.2 20020527 (experimental)
>Organization:
>Environment:
Mandrake 8.2 x86
Configured with: ../gcc/configure --prefix=/usr/local/gcc/ --enable-languages=c
>Description:
I found this when trying to compile lbreakout 2.2.2

It only triggers when using -O or higher together with -march=pentium3, -march=pentium2 or -march=athlon (and perhaps others)

a.c: In function `circle_intersect':
a.c:10: Internal compiler error in ix86_secondary_memory_needed, at config/i386/i386.c:13513
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://www.gnu.org/software/gcc/bugs.html> for instructions.

I'm not sure if
Vector delta = { pos.x - m.x, pos.y - m.y };
is legal, but it doesn't generate any warnings.
>How-To-Repeat:

>Fix:
Replace
Vector delta = { pos.x - m.x, pos.y - m.y };
with
Vector delta;
delta.x = pos.x - m.x;
delta.y = pos.y - m.y;
>Release-Note:
>Audit-Trail:
>Unformatted:
----gnatsweb-attachment----
Content-Type: application/octet-stream; name="a.i"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="a.i"

IyAxICJhLmMiCiMgMSAiPGJ1aWx0LWluPiIKIyAxICI8Y29tbWFuZCBsaW5lPiIKIyAxICJhLmMi
CnR5cGVkZWYgc3RydWN0IHsKICAgIGZsb2F0IHgsIHk7Cn0gVmVjdG9yOwoKaW50IGNpcmNsZV9p
bnRlcnNlY3QoIFZlY3RvciBtLCBWZWN0b3IgcG9zKQp7CiAgICBWZWN0b3IgZGVsdGEgPSB7IHBv
cy54IC0gbS54LCBwb3MueSAtIG0ueSB9OwoKICAgIHJldHVybiBkZWx0YS54Owp9Cg==