On Tue, Sep 9, 2014 at 12:21 PM, Bill Schmidt <wschm...@linux.vnet.ibm.com> wrote: > Hi, > > When collecting my list of invalid UNSPECs for the analyze_swaps pass, I > missed UNSPEC_VSX_CVDPSPN (which looked too much like > UNSPEC_VSX_CVSPDPN). This patch adds it to the list. I have an idea > for allowing it to be used, but won't have time to work on it now, so I > added some commentary to document it. > > This causes the swaps-p8-15.c test to no longer be optimized, so I've > removed the optimization check there for now, with a comment on > reinstating it later. (I also removed an unused typedef that I happened > to notice while I was in there.) > > Bootstrapped and tested on powerpc64le-unknown-linux-gnu with no > regressions. Ok for trunk? > > Thanks, > Bill > > > [gcc] > > 2014-09-09 Bill Schmidt <wschm...@linux.vnet.ibm.com> > > * config/rs6000/rs6000.c (rtx_is_swappable_p): Add > UNSPEC_VSX_CVDPSPN as an unswappable operand, and add commentary > on how to make it legal in future. > > [gcc/testsuite] > > 2014-09-09 Bill Schmidt <wschm...@linux.vnet.ibm.com> > > * gcc.target/powerpc/swaps-p8-15.c: Remove scan-assembler-not for > xxpermdi and add commentary about adding it back later; remove > unused typedef.
Okay. Thanks, David