[PATCH] Fix sse2_loadlpd splitter (PR target/52736)

Uros Bizjak ubizjak@gmail.com
Wed Mar 28 06:05:00 GMT 2012


On Tue, Mar 27, 2012 at 10:36 PM, Jakub Jelinek <jakub@redhat.com> wrote:

> As the following testcase shows, the sse2_loadlpd splitter when
> the destination (and at the same operand from which the second
> DFmode element is taken) is a MEM and the other DFmode operand
> is a register stores into wrong part of memory, it does what
> the sse2_loadhpd splitter does, while it should overwrite the
> first half of the vector and keep the second half preserved.
>
> Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
> ok for trunk/4.7/4.6?
>
> 2012-03-27  Jakub Jelinek  <jakub@redhat.com>
>
>        PR target/52736
>        * config/i386/sse.md (sse2_loadlpd splitter): Use offset 0
>        instead of 8 in adjust_address.
>
>        * gcc.target/i386/pr52736.c: New test.

OK everywhere.

Thanks,
Uros.



More information about the Gcc-patches mailing list