On Fri, Aug 26, 2016 at 08:24:40PM -0500, Segher Boessenkool wrote:
> On Fri, Aug 26, 2016 at 03:29:50PM -0400, Michael Meissner wrote:
> > These patches add more tests to the PowerPC vector initialization tests.
> > Four
> > of the tests added (#4, #5, #8, and #9) just try to do a bunch of vector
> > initializations for the different vector type (char, short, float, and
> > double).
> >
> > These other patches (#6, #7) test the code generated in paches #5 and #6.
> >
> > I have run tese tests on a big endian power7 system (with both 32-bt and
> > 64-bit
> > tests runs), a big endian power8 system (just 64-bit tests), and a little
> > endian power8 system. There were no regressions. As these tests ok to
> > install?
>
> This is okay for trunk; one comment:
>
> > +/* { dg-final { scan-assembler "mtvsrd" } } */
>
> This also matches mtvsrdd; if you don't want that, you can avoid it by
> writing it as {\mmtvsrd\M} (the {} instead of "" to avoid toothpickeritus).
In this case, mtvsrd and mtvsrdd are both fine. However, given the test has an
explicit -mcpu=power8, it should never see mtvsrdd.
--
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797
