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).