This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug target/36827] [4.3/4.4 Regression] newlib:libm/math/k_rem_pio2.c fails in reload
- From: "joseph at codesourcery dot com" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 16 Jul 2008 20:25:03 -0000
- Subject: [Bug target/36827] [4.3/4.4 Regression] newlib:libm/math/k_rem_pio2.c fails in reload
- References: <bug-36827-681@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- 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