[PATCH] Fix PR target/6512
Jakub Jelinek
jakub@redhat.com
Tue Apr 30 15:54:00 GMT 2002
On Tue, Apr 30, 2002 at 03:12:35PM -0700, David S. Miller wrote:
> From: Richard Henderson <rth@redhat.com>
> Date: Tue, 30 Apr 2002 12:31:12 -0700
>
> On Tue, Apr 30, 2002 at 02:19:05PM +0200, Jakub Jelinek wrote:
> > I've tried patch P3 (changing e -> f constraints for e <- o and o <- e
> > alternatives), but it did work at all, for the above insn
> > reload just tried (set %f40 (mem ())); (set %f40 %f40).
>
> Means you're missing a test in SECONDARY_RELOAD_NEEDED.
>
> The P3 test is more correct. The aligned case is handled by
> the e/W alternative.
>
> Please note, this is almost surely the same bug as shown by
> PR target/5628
>
> If you fix PR target/6512 properly, you'll most likely end up
> fixing PR target/5628 as well.
I've just tested it and the patch I'm bootstrapping ATM for target/6512
(P3 + corresponding SECONDARY_*_CLASS changes) fixes target/5628 too.
Jakub
More information about the Gcc-patches
mailing list