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
>
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
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
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