public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug rtl-optimization/57810] New: Waste work in validate_const_int()
@ 2013-07-03 19:00 pchang9 at cs dot wisc.edu
2013-07-03 20:26 ` [Bug rtl-optimization/57810] Wasted " pchang9 at cs dot wisc.edu
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-03 19:00 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57810
Bug ID: 57810
Summary: Waste work in validate_const_int()
Product: gcc
Version: 4.9.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: rtl-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: pchang9 at cs dot wisc.edu
CC: nistor1 at illinois dot edu
The problem appears in revision 200588 in version 4.9.
In method "validate_const_int()" in gcc/read-rtl.c, the loop on line 804 should
break immediately after "valid" is set to "0". All the iterations after "valid"
set to "0" do not perform any useful work, at best they just set "valid" again
to "0".
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug rtl-optimization/57810] Wasted work in validate_const_int()
2013-07-03 19:00 [Bug rtl-optimization/57810] New: Waste work in validate_const_int() pchang9 at cs dot wisc.edu
@ 2013-07-03 20:26 ` pchang9 at cs dot wisc.edu
2013-07-17 16:48 ` pchang9 at cs dot wisc.edu
2013-07-18 16:11 ` law at redhat dot com
2 siblings, 0 replies; 4+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-03 20:26 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57810
--- Comment #1 from Po-Chun Chang <pchang9 at cs dot wisc.edu> ---
Created attachment 30440
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30440&action=edit
Suggested patch
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug rtl-optimization/57810] Wasted work in validate_const_int()
2013-07-03 19:00 [Bug rtl-optimization/57810] New: Waste work in validate_const_int() pchang9 at cs dot wisc.edu
2013-07-03 20:26 ` [Bug rtl-optimization/57810] Wasted " pchang9 at cs dot wisc.edu
@ 2013-07-17 16:48 ` pchang9 at cs dot wisc.edu
2013-07-18 16:11 ` law at redhat dot com
2 siblings, 0 replies; 4+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-17 16:48 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57810
--- Comment #2 from Po-Chun Chang <pchang9 at cs dot wisc.edu> ---
Patch sent to gcc-patches@:
http://gcc.gnu.org/ml/gcc-patches/2013-07/msg00662.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug rtl-optimization/57810] Wasted work in validate_const_int()
2013-07-03 19:00 [Bug rtl-optimization/57810] New: Waste work in validate_const_int() pchang9 at cs dot wisc.edu
2013-07-03 20:26 ` [Bug rtl-optimization/57810] Wasted " pchang9 at cs dot wisc.edu
2013-07-17 16:48 ` pchang9 at cs dot wisc.edu
@ 2013-07-18 16:11 ` law at redhat dot com
2 siblings, 0 replies; 4+ messages in thread
From: law at redhat dot com @ 2013-07-18 16:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57810
Jeffrey A. Law <law at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
CC| |law at redhat dot com
Resolution|--- |FIXED
--- Comment #3 from Jeffrey A. Law <law at redhat dot com> ---
Fixed on trunk.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-07-18 16:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-03 19:00 [Bug rtl-optimization/57810] New: Waste work in validate_const_int() pchang9 at cs dot wisc.edu
2013-07-03 20:26 ` [Bug rtl-optimization/57810] Wasted " pchang9 at cs dot wisc.edu
2013-07-17 16:48 ` pchang9 at cs dot wisc.edu
2013-07-18 16:11 ` law at redhat dot com
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).