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