------- Comment #12 from joseph at codesourcery dot com 2008-07-16 20:25 ------- Subject: Re: [4.3/4.4 Regression] newlib:libm/math/k_rem_pio2.c fails in reload
On Wed, 16 Jul 2008, dj at redhat dot com wrote: > Last night's builds showed a flaw in the patch. The "++rii" address > created by m32c_legitimize_reload_address is *not* legitimate, it's > used as a marker for addresses that need to be reloaded into an > address register. If I fix m32c_print_operand() to print those, it > ends up printing an address that exceeds the displacement range of the > FB register: Try changing the "reloaded != 1" condition in find_reloads_subreg_address to "reloaded == 0" (as a replacement for my m32c patch, and probably for the SH patch as well). I hope that should avoid these problems in the cases where LEGITIMIZE_RELOAD_ADDRESS does something like this, and I've verified the original ARM testcase that motivated my reload change still works with that change. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36827
