Re: [PATCH, rs6000] Add Power 8 support to vec_revb

2017-10-17 Thread Segher Boessenkool
Hi Carl, On Tue, Oct 17, 2017 at 09:56:43AM -0700, Carl Love wrote: > gcc/ChangeLog: > > 2017-10-17 Carl Love > > * config/rs6000/rs6000-c.c (P8V_BUILTIN_VEC_REVB): > Add power 8 definitions for the builtin instances. > (P9V_BUILTIN_VEC_REVB): Remove the power 9 instance >

Re: [PATCH, rs6000] Add Power 8 support to vec_revb

2017-10-17 Thread Carl Love
GCC maintainers: I have addressed the issues with the vec_revb patch mentioned by Segher. I have retested the updated patch on: powerpc64-unknown-linux-gnu (Power 8 BE), powerpc64le-unknown-linux-gnu (Power 8 LE), powerpc64le-unknown-linux-gnu (Power 9 LE) without regressions. Please le

Re: [PATCH, rs6000] Add Power 8 support to vec_revb

2017-10-09 Thread Segher Boessenkool
Hi Carl, On Mon, Oct 09, 2017 at 08:36:57AM -0700, Carl Love wrote: > *gcc.target/powerpc/builtins-revb-runnable.c: New > runnable test file for the vec_revb builtin. (Missing space after the asterisk). > diff --git a/gcc/config/rs6000/vsx.md b/gcc/config/rs6000/vsx.md > index b47eea

[PATCH, rs6000] Add Power 8 support to vec_revb

2017-10-09 Thread Carl Love
GCC maintainers: The following patch add Power8 vec_revb() builtin support to the existing Power 9 support. The Power ISA specification says the vec_revb() builtin is to be supported on both Power 8 and Power 9. There are new instructions in Power 9 that support the vec_revb() functionality. So