public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/57811] New: Waste work in find_reloads()
@ 2013-07-03 19:08 pchang9 at cs dot wisc.edu
2013-07-03 20:29 ` [Bug middle-end/57811] Wasted " pchang9 at cs dot wisc.edu
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-03 19:08 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57811
Bug ID: 57811
Summary: Waste work in find_reloads()
Product: gcc
Version: 4.9.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: middle-end
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 "find_reloads()" in gcc/reload.c, the loop on line 3324 should break
immediately after "badop" is set to "1". All the iterations after "badop" set
to "1" do not perform any useful work, at best they just set "badop" again to
"1".
Similar problem also appears in the same method "find_reloads()" in line 4641.
The loop should break immediately after "ok" is set to "0".
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug middle-end/57811] Wasted work in find_reloads()
2013-07-03 19:08 [Bug middle-end/57811] New: Waste work in find_reloads() pchang9 at cs dot wisc.edu
@ 2013-07-03 20:29 ` pchang9 at cs dot wisc.edu
2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-03 20:29 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57811
--- Comment #1 from Po-Chun Chang <pchang9 at cs dot wisc.edu> ---
Created attachment 30442
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30442&action=edit
Suggested patch for loop at line 3324
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug middle-end/57811] Wasted work in find_reloads()
2013-07-03 19:08 [Bug middle-end/57811] New: Waste work in find_reloads() pchang9 at cs dot wisc.edu
2013-07-03 20:29 ` [Bug middle-end/57811] Wasted " pchang9 at cs dot wisc.edu
2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
@ 2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
2013-07-23 15:59 ` law at redhat dot com
3 siblings, 0 replies; 5+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-03 20:43 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57811
--- Comment #3 from Po-Chun Chang <pchang9 at cs dot wisc.edu> ---
Created attachment 30445
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30445&action=edit
Suggested patch for loop at line 4641
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug middle-end/57811] Wasted work in find_reloads()
2013-07-03 19:08 [Bug middle-end/57811] New: Waste work in find_reloads() pchang9 at cs dot wisc.edu
2013-07-03 20:29 ` [Bug middle-end/57811] Wasted " pchang9 at cs dot wisc.edu
@ 2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
2013-07-23 15:59 ` law at redhat dot com
3 siblings, 0 replies; 5+ messages in thread
From: pchang9 at cs dot wisc.edu @ 2013-07-03 20:43 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57811
--- Comment #2 from Po-Chun Chang <pchang9 at cs dot wisc.edu> ---
Created attachment 30444
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30444&action=edit
Suggested patch for loop at line 3324
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug middle-end/57811] Wasted work in find_reloads()
2013-07-03 19:08 [Bug middle-end/57811] New: Waste work in find_reloads() pchang9 at cs dot wisc.edu
` (2 preceding siblings ...)
2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
@ 2013-07-23 15:59 ` law at redhat dot com
3 siblings, 0 replies; 5+ messages in thread
From: law at redhat dot com @ 2013-07-23 15:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57811
Jeffrey A. Law <law at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
CC| |law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law <law at redhat dot com> ---
Fixed in mainline.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-07-23 15:59 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-03 19:08 [Bug middle-end/57811] New: Waste work in find_reloads() pchang9 at cs dot wisc.edu
2013-07-03 20:29 ` [Bug middle-end/57811] Wasted " pchang9 at cs dot wisc.edu
2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
2013-07-03 20:43 ` pchang9 at cs dot wisc.edu
2013-07-23 15:59 ` 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).