public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
* Re: optimization/9037: [3.3 regression] -fssa-dce crashes with simple code
@ 2003-03-24 0:16 Steven Bosscher
0 siblings, 0 replies; 5+ messages in thread
From: Steven Bosscher @ 2003-03-24 0:16 UTC (permalink / raw)
To: steven; +Cc: gcc-prs
The following reply was made to PR optimization/9037; it has been noted by GNATS.
From: Steven Bosscher <s.bosscher@student.tudelft.nl>
To: gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org, steven@gcc.gnu.org,
renau@cs.uiuc.edu, gcc-prs@gcc.gnu.org, mark@codesourcery.com,
gcc-patches@gcc.gnu.org
Cc:
Subject: Re: optimization/9037: [3.3 regression] -fssa-dce crashes with simple
code
Date: Mon, 24 Mar 2003 00:34:55 +0100
This is a multi-part message in MIME format.
--------------060208020800070202080801
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=9037
A patch for this from Jose Renau (who is also the
reporter of this problem) was applied almost three
months ago on mainline. Here's a backport of the
patch. OK for 3.3 if it passes regression tests?
I don't know what the correct ChangeLog entry should
be, so I just copy-pasted the entry from the mainline
ChangeLog. Is there a convention for backported
changes?
Greetz
Steven
2002-12-26 Jose Renau <renau@cs.uiuc.edu>
* ssa-dce.c (EXECUTE_IF_UNNECESSARY): Verify INSN is an
INSN_P before checking to see if it is dead.
(mark_all_insn_unnecessary): Similarly.
(ssa_eliminate_dead_code): Similarly.
* rtl.h (struct rtx_def): Update comments for in_struct usage
in dead code elimination pass.
(INSN_DEAD_CODE_P): Allow JUMP_INSN and CALL_INSN as well.
--------------060208020800070202080801
Content-Type: application/x-java-vm;
name="patch"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
filename="patch"
SW5kZXg6IHJ0bC5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvZ2NjL2djYy9nY2Mv
cnRsLmgsdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuMzc1LjIuMgpkaWZmIC1jIC0zIC1wIC1y
MS4zNzUuMi4yIHJ0bC5oCioqKiBydGwuaAkyMyBNYXIgMjAwMyAyMDoxNzoxOSAtMDAwMAkx
LjM3NS4yLjIKLS0tIHJ0bC5oCTIzIE1hciAyMDAzIDIzOjIxOjIxIC0wMDAwCioqKioqKioq
KioqKioqKiBkbyB7CQkJCVwKKioqIDU3OCw1ODQgKioqKgogIC8qIDEgaWYgUlRYIGlzIGFu
IGluc24gdGhhdCBpcyBkZWFkIGNvZGUuICBWYWxpZCBvbmx5IGZvciBkZWFkLWNvZGUKICAg
ICBlbGltaW5hdGlvbiBwaGFzZS4gICovCiAgI2RlZmluZSBJTlNOX0RFQURfQ09ERV9QKFJU
WCkJCQkJCQlcCiEgICAoUlRMX0ZMQUdfQ0hFQ0sxKCJJTlNOX0RFQURfQ09ERV9QIiwgKFJU
WCksIElOU04pLT5pbl9zdHJ1Y3QpCiAgCiAgLyogMSBpZiBSVFggaXMgYW4gaW5zbiBpbiBh
IGRlbGF5IHNsb3QgYW5kIGlzIGZyb20gdGhlIHRhcmdldCBvZiB0aGUgYnJhbmNoLgogICAg
IElmIHRoZSBicmFuY2ggaW5zbiBoYXMgSU5TTl9BTk5VTExFRF9CUkFOQ0hfUCBzZXQsIHRo
aXMgaW5zbiBzaG91bGQgb25seSBiZQotLS0gNTc4LDU4NCAtLS0tCiAgLyogMSBpZiBSVFgg
aXMgYW4gaW5zbiB0aGF0IGlzIGRlYWQgY29kZS4gIFZhbGlkIG9ubHkgZm9yIGRlYWQtY29k
ZQogICAgIGVsaW1pbmF0aW9uIHBoYXNlLiAgKi8KICAjZGVmaW5lIElOU05fREVBRF9DT0RF
X1AoUlRYKQkJCQkJCVwKISAgIChSVExfRkxBR19DSEVDSzMoIklOU05fREVBRF9DT0RFX1Ai
LCAoUlRYKSwgSU5TTiwgQ0FMTF9JTlNOLCBKVU1QX0lOU04pLT5pbl9zdHJ1Y3QpCiAgCiAg
LyogMSBpZiBSVFggaXMgYW4gaW5zbiBpbiBhIGRlbGF5IHNsb3QgYW5kIGlzIGZyb20gdGhl
IHRhcmdldCBvZiB0aGUgYnJhbmNoLgogICAgIElmIHRoZSBicmFuY2ggaW5zbiBoYXMgSU5T
Tl9BTk5VTExFRF9CUkFOQ0hfUCBzZXQsIHRoaXMgaW5zbiBzaG91bGQgb25seSBiZQpJbmRl
eDogc3NhLWRjZS5jCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvZ2NjL2djYy9nY2Mv
c3NhLWRjZS5jLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjIyCmRpZmYgLWMgLTMgLXAgLXIx
LjIyIHNzYS1kY2UuYwoqKiogc3NhLWRjZS5jCTIwIEp1biAyMDAyIDE3OjUxOjA0IC0wMDAw
CTEuMjIKLS0tIHNzYS1kY2UuYwkyMyBNYXIgMjAwMyAyMzoyMToyMyAtMDAwMAoqKioqKioq
KioqKioqKiogc3RhdGljIHZvaWQgbWFya19hbGxfaW5zbl91bm5lY2Vzc2FyeQoqKiogMTMz
LDE0MCAqKioqCiAgICBydHggSU5TTjsJCQkJCQkJXAogIAkJCQkJCQkJXAogICAgZm9yIChJ
TlNOID0gZ2V0X2luc25zICgpOyBJTlNOICE9IE5VTExfUlRYOyBJTlNOID0gTkVYVF9JTlNO
IChJTlNOKSkJXAohICAgICBpZiAoSU5TTl9ERUFEX0NPREVfUCAoSU5TTikpIHsJCQkJXAoh
ICAgICAgIENPREU7CQkJCQkJCVwKICAgICAgfQkJCQkJCQkJXAogIH0KICAvKiBGaW5kIHRo
ZSBsYWJlbCBiZWdpbm5pbmcgYmxvY2sgQkIuICAqLwotLS0gMTMzLDE0MiAtLS0tCiAgICBy
dHggSU5TTjsJCQkJCQkJXAogIAkJCQkJCQkJXAogICAgZm9yIChJTlNOID0gZ2V0X2luc25z
ICgpOyBJTlNOICE9IE5VTExfUlRYOyBJTlNOID0gTkVYVF9JTlNOIChJTlNOKSkJXAohICAg
ICBpZiAoIElOU05fUCAoaW5zbikgKSB7CQkJCQlcCiEgICAgICAgaWYgKElOU05fREVBRF9D
T0RFX1AgKElOU04pKSB7CQkJCVwKISAgICAgICAgIENPREU7CQkJCQkJCVwKISAgICAgICB9
CQkJCQkJCQlcCiAgICAgIH0JCQkJCQkJCVwKICB9CiAgLyogRmluZCB0aGUgbGFiZWwgYmVn
aW5uaW5nIGJsb2NrIEJCLiAgKi8KKioqKioqKioqKioqKioqIHN0YXRpYyB2b2lkCioqKiA0
NDYsNDUzICoqKioKICBtYXJrX2FsbF9pbnNuX3VubmVjZXNzYXJ5ICgpCiAgewogICAgcnR4
IGluc247CiEgICBmb3IgKGluc24gPSBnZXRfaW5zbnMgKCk7IGluc24gIT0gTlVMTF9SVFg7
IGluc24gPSBORVhUX0lOU04gKGluc24pKQohICAgICBLSUxMX0lOU04gKGluc24pOwogIH0K
ICAKICAvKiBGaW5kIHRoZSBsYWJlbCBiZWdpbm5pbmcgYmxvY2sgQkIsIGFkZGluZyBvbmUg
aWYgbmVjZXNzYXJ5LiAgKi8KLS0tIDQ0OCw0NTggLS0tLQogIG1hcmtfYWxsX2luc25fdW5u
ZWNlc3NhcnkgKCkKICB7CiAgICBydHggaW5zbjsKISAgIGZvciAoaW5zbiA9IGdldF9pbnNu
cyAoKTsgaW5zbiAhPSBOVUxMX1JUWDsgaW5zbiA9IE5FWFRfSU5TTiAoaW5zbikpIHsKISAg
ICAgaWYgKCBJTlNOX1AgKGluc24pKQohICAgICAgIEtJTExfSU5TTiAoaW5zbik7CiEgICB9
CiEgICAKICB9CiAgCiAgLyogRmluZCB0aGUgbGFiZWwgYmVnaW5uaW5nIGJsb2NrIEJCLCBh
ZGRpbmcgb25lIGlmIG5lY2Vzc2FyeS4gICovCioqKioqKioqKioqKioqKiBzc2FfZWxpbWlu
YXRlX2RlYWRfY29kZSAoKQoqKiogNTIwLDUyNiAqKioqCiAgCiAgICAvKiBGaW5kIGluaGVy
ZW50bHkgbmVjZXNzYXJ5IGluc3RydWN0aW9ucy4gICovCiAgICBmb3IgKGluc24gPSBnZXRf
aW5zbnMgKCk7IGluc24gIT0gTlVMTF9SVFg7IGluc24gPSBORVhUX0lOU04gKGluc24pKQoh
ICAgICBpZiAoZmluZF9pbmhlcmVudGx5X25lY2Vzc2FyeSAoaW5zbikpCiAgICAgICAgewog
IAlSRVNVUlJFQ1RfSU5TTiAoaW5zbik7CiAgCVZBUlJBWV9QVVNIX1JUWCAodW5wcm9jZXNz
ZWRfaW5zdHJ1Y3Rpb25zLCBpbnNuKTsKLS0tIDUyNSw1MzEgLS0tLQogIAogICAgLyogRmlu
ZCBpbmhlcmVudGx5IG5lY2Vzc2FyeSBpbnN0cnVjdGlvbnMuICAqLwogICAgZm9yIChpbnNu
ID0gZ2V0X2luc25zICgpOyBpbnNuICE9IE5VTExfUlRYOyBpbnNuID0gTkVYVF9JTlNOIChp
bnNuKSkKISAgICAgaWYgKGZpbmRfaW5oZXJlbnRseV9uZWNlc3NhcnkgKGluc24pICYmIElO
U05fUCAoaW5zbikpCiAgICAgICAgewogIAlSRVNVUlJFQ1RfSU5TTiAoaW5zbik7CiAgCVZB
UlJBWV9QVVNIX1JUWCAodW5wcm9jZXNzZWRfaW5zdHJ1Y3Rpb25zLCBpbnNuKTsKKioqKioq
KioqKioqKioqIHNzYV9lbGltaW5hdGVfZGVhZF9jb2RlICgpCioqKiA3MjMsNzMwICoqKioK
ICAJfQogICAgICB9CiAgICAvKiBSZWxlYXNlIGFsbG9jYXRlZCBtZW1vcnkuICAqLwohICAg
Zm9yIChpbnNuID0gZ2V0X2luc25zICgpOyBpbnNuICE9IE5VTExfUlRYOyBpbnNuID0gTkVY
VF9JTlNOIChpbnNuKSkKISAgICAgUkVTVVJSRUNUX0lOU04gKGluc24pOwogICAgaWYgKFZB
UlJBWV9BQ1RJVkVfU0laRSAodW5wcm9jZXNzZWRfaW5zdHJ1Y3Rpb25zKSAhPSAwKQogICAg
ICBhYm9ydCAoKTsKICAgIGNvbnRyb2xfZGVwZW5kZW50X2Jsb2NrX3RvX2VkZ2VfbWFwX2Zy
ZWUgKGNkYnRlKTsKLS0tIDcyOCw3MzggLS0tLQogIAl9CiAgICAgIH0KICAgIC8qIFJlbGVh
c2UgYWxsb2NhdGVkIG1lbW9yeS4gICovCiEgICBmb3IgKGluc24gPSBnZXRfaW5zbnMgKCk7
IGluc24gIT0gTlVMTF9SVFg7IGluc24gPSBORVhUX0lOU04gKGluc24pKSB7CiEgICAgIGlm
ICggSU5TTl9QIChpbnNuKSkKISAgICAgICBSRVNVUlJFQ1RfSU5TTiAoaW5zbik7CiEgICB9
CiEgICAKICAgIGlmIChWQVJSQVlfQUNUSVZFX1NJWkUgKHVucHJvY2Vzc2VkX2luc3RydWN0
aW9ucykgIT0gMCkKICAgICAgYWJvcnQgKCk7CiAgICBjb250cm9sX2RlcGVuZGVudF9ibG9j
a190b19lZGdlX21hcF9mcmVlIChjZGJ0ZSk7Cg==
--------------060208020800070202080801--
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: optimization/9037: [3.3 regression] -fssa-dce crashes with simple code
@ 2003-03-28 11:47 steven
0 siblings, 0 replies; 5+ messages in thread
From: steven @ 2003-03-28 11:47 UTC (permalink / raw)
To: gcc-bugs, gcc-prs, nobody, renau
Synopsis: [3.3 regression] -fssa-dce crashes with simple code
State-Changed-From-To: analyzed->closed
State-Changed-By: steven
State-Changed-When: Fri Mar 28 11:30:07 2003
State-Changed-Why:
Was not tree-ssa specific. Fixed on mainline. Closed since this will not be fixed for 3.3 anyway.
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=9037
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: optimization/9037: [3.3 regression] -fssa-dce crashes with simple code
@ 2003-03-24 8:31 steven
0 siblings, 0 replies; 5+ messages in thread
From: steven @ 2003-03-24 8:31 UTC (permalink / raw)
To: gcc-bugs, gcc-prs, nobody, renau, steven
Synopsis: [3.3 regression] -fssa-dce crashes with simple code
Responsible-Changed-From-To: steven->unassigned
Responsible-Changed-By: steven
Responsible-Changed-When: Mon Mar 24 07:55:30 2003
Responsible-Changed-Why:
"We don't care about SSA for 3.3"
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=9037
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: optimization/9037: [3.3 regression] -fssa-dce crashes with simple code
@ 2003-03-24 6:36 Mark Mitchell
0 siblings, 0 replies; 5+ messages in thread
From: Mark Mitchell @ 2003-03-24 6:36 UTC (permalink / raw)
To: steven; +Cc: gcc-prs
The following reply was made to PR optimization/9037; it has been noted by GNATS.
From: Mark Mitchell <mark@codesourcery.com>
To: Steven Bosscher <s.bosscher@student.tudelft.nl>
Cc: gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org, steven@gcc.gnu.org,
renau@cs.uiuc.edu, gcc-prs@gcc.gnu.org, gcc-patches@gcc.gnu.org
Subject: Re: optimization/9037: [3.3 regression] -fssa-dce crashes with
simple code
Date: 23 Mar 2003 21:41:58 -0800
On Sun, 2003-03-23 at 15:34, Steven Bosscher wrote:
> http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=9037
>
> A patch for this from Jose Renau (who is also the
> reporter of this problem) was applied almost three
> months ago on mainline. Here's a backport of the
> patch. OK for 3.3 if it passes regression tests?
No. We don't care about SSA for 3.3.
Thanks,
--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: optimization/9037: [3.3 regression] -fssa-dce crashes with simple code
@ 2003-03-23 23:36 steven
0 siblings, 0 replies; 5+ messages in thread
From: steven @ 2003-03-23 23:36 UTC (permalink / raw)
To: gcc-bugs, gcc-prs, nobody, renau, steven
Old Synopsis: [3.3 regression] [tree-ssa] -fssa-dce crashes with simple code
New Synopsis: [3.3 regression] -fssa-dce crashes with simple code
Responsible-Changed-From-To: unassigned->steven
Responsible-Changed-By: steven
Responsible-Changed-When: Sun Mar 23 23:20:20 2003
Responsible-Changed-Why:
Has nothing to do with tree-ssa.
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=9037
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2003-03-28 11:30 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-03-24 0:16 optimization/9037: [3.3 regression] -fssa-dce crashes with simple code Steven Bosscher
-- strict thread matches above, loose matches on Subject: below --
2003-03-28 11:47 steven
2003-03-24 8:31 steven
2003-03-24 6:36 Mark Mitchell
2003-03-23 23:36 steven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).