On 07/17/2013 10:41 AM, pcha...@cs.wisc.edu wrote:
Hi,
The problem appears in revision 200945 in version 4.9. I attached
a one-line patch that fixes it. I also reported this problem at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57810 .
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".
It may seem a bit of overkill for these patchs, but did you bootstrap
and/or regression test this or your prior patch?
Our policies require bootstrap and regression tests to help ensure the
quality of the sourcebase. I'll go ahead and bootstrap your two changes
for the sake of completeness (they both look obviously correct to me).
Thanks,
Jeff