On Thu, Nov 21, 2013 at 7:27 PM, Bill Schmidt
<wschm...@linux.vnet.ibm.com> wrote:
> Hi,
>
> This patch fixes two issues to allow correct compilation of
> gcc.dg/torture/vec-cvt-1.c in little endian mode.  The first reverts a
> change in three patterns in vector.md.  This is from an early patch that
> preceded the general fix for vector permutes.  As a consequence we ended
> up swapping the input arguments twice.  So we can simplify the code here
> and have it operate the same for big and little endian.
>
> The other issue corrects a scenario where I managed to check for
> endianness twice, with the effect that the code acts the same for both
> big and little endian when it shouldn't.
>
> Bootstrapped and tested on powerpc64{,le}-unknown-linux-gnu with no
> regressions.  Is this ok for trunk?
>
> Thanks,
> Bill
>
>
> 2013-11-21  Bill Schmidt  <wschm...@vnet.ibm.com>
>
>         * config/rs6000/vector.md (vec_pack_trunc_v2df): Revert previous
>         little endian change.
>         (vec_pack_sfix_trunc_v2df): Likewise.
>         (vec_pack_ufix_trunc_v2df): Likewise.
>         * config/rs6000/rs6000.c (rs6000_expand_interleave): Correct
>         double checking of endianness.

Okay.

Thanks, David

Reply via email to