Hi!

On Wed, Jan 24, 2018 at 10:46:02AM -0600, Will Schmidt wrote:
> 2018-01-24  Will Schmidt  <will_schm...@vnet.ibm.com>
> 
>       * gcc.target/powerpc/fold-vec-abs-int.c:  remove scan-assembler stanzas.

Only one space after : and start with a cap.

>       * gcc.target/powerpc/fold-vec-abs-int-fwrap.c:  Same.
>       * gcc.target/powerpc/fold-vec-abs-int.p7.c: New.
>       * gcc.target/powerpc/fold-vec-abs-int.p8.c: New.
>       * gcc.target/powerpc/fold-vec-abs-int.p9.c: New.
>       * gcc.target/powerpc/fold-vec-abs-int-fwrapv.p7.c: New.
>       * gcc.target/powerpc/fold-vec-abs-int-fwrapv.p8.c: New.
>       * gcc.target/powerpc/fold-vec-abs-int-fwrapv.p9.c: New.
>       * gcc.target/powerpc/fold-vec-abs-longlong.c:  remove scan-assembler 
> stanzas.
>       * gcc.target/powerpc/fold-vec-abs-longlong-fwrap.c:  Same.
>       * gcc.target/powerpc/fold-vec-abs-longlong.p7.c: New.
>       * gcc.target/powerpc/fold-vec-abs-longlong.p8.c: New.
>       * gcc.target/powerpc/fold-vec-abs-longlong.p9.c: New.
>       * gcc.target/powerpc/fold-vec-abs-longlong-fwrapv.p7.c: New.
>       * gcc.target/powerpc/fold-vec-abs-longlong-fwrapv.p8.c: New.
>       * gcc.target/powerpc/fold-vec-abs-longlong-fwrapv.p9.c: New.
>       * gcc.target/powerpc/fold-vec-abs-short.c:  Add xxspltib to valid 
> instruction list.
>       * gcc.target/powerpc/fold-vec-abs-short-fwrapv.c:  Same.

More of that in the other entries, please check all.

This looks fine.  One question: should we also test expected code
generation for systems that only have AltiVec, not VSX?  So something
older than p7?  That goes for this whole series of course.

But, okay for trunk either way.  Thanks!


Segher

Reply via email to