This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: rs6000: Some fixes for bswapdi
On Tue, Sep 8, 2009 at 3:47 PM, Segher
Boessenkool<segher@kernel.crashing.org> wrote:
> For a memory reference like [A+B], the splitters for bswapdi should
> split this to SI [A+B] and [(A+4)+B]; the code however does [B] and
> [(A+4)+B]. ?This actually ICEd (on a default Linux kernel build)
> because B was r0.
>
> Also changes some of the register constraints which used "b" while
> they could be "r".
>
> Also fixes a few instances where TARGET_32BIT used 64-bit math for
> addresses.
>
> Bootstrapped and tested on powerpc64-linux, no regressions. ?Okay
> to apply?
>
> ? ? ? ?2009-09-08 ?Segher Boessenkool ?<segher@kernel.crashing.org>
>
> ? ? ? ? ? ? ? ?* config/rs6000/rs6000.md (bswapdi2_64bit): Fix
> ? ? ? ? ? ? ? ?unnecessarily stringent constraints. ?Fix address
> ? ? ? ? ? ? ? ?calculation in the splitters.
Segher,
This patch is okay.
Thanks, David