Hi! On Fri, May 04, 2018 at 07:59:37AM -0700, Carl Love wrote: > > Why does the patch remove these two vec_sel? If that is wanted, the > > changelog should mention this.
> No clue why I removed the vec_sel test. Been sitting on the testsuite > patch set too long, I don't recall. I put it back in and tested it. > Then noticed there wasn't any tests for the xxsel instruction which the > vec_sel test generates. I added an instruction count test to both the > BE and LE tests. I found the BE test needed some updating as well. > The revised patch is given below. I retested on: > > powerpc64le-unknown-linux-gnu (Power 8 LE) > powerpc64-unknown-linux-gnu (Power 8 BE) > powerpc64le-unknown-linux-gnu (Power 9 LE). > > Please let me know if everything looks OK now. Thanks. Looks good to me, okay for trunk. Thanks! (I'll find out soon enough if it works on power7 as well :-) ) Segher > 2018-05-03 Carl Love <c...@us.ibm.com> > * gcc.target/powerpc/vsx-vector-6.h (foo): Add test for vec_max, > vec_trunc. > * gcc.target/powerpc/vsx-vector-6-le.c (dg-final): Update xvcmpeqdp, > xvcmpgtdp, xvcmpgedp counts. Add xxsel counts. > * gcc.target/powerpc/vsx-vector-6-be.c (dg-final): Update xvcmpgtdp, > xvcmpgedp counts. Add xxsel counts.