Re: [PATCH v2, rs6000] Fix implementation of vec_pack (vector double, vector double) built-in function

2018-06-22 Thread Kelvin Nilsen
Thanks for feedback. It turns out that the vmrgew and vmrgow instructions require power 8. After coordinating with Segher on minor refinements to the test cases, I have committed the patch as quoted below to the trunk. On 6/19/18 5:37 PM, Segher Boessenkool wrote: > Hi! > > On Tue, Jun 19,

Re: [PATCH v2, rs6000] Fix implementation of vec_pack (vector double, vector double) built-in function

2018-06-19 Thread Segher Boessenkool
Hi! On Tue, Jun 19, 2018 at 01:37:51PM -0500, Kelvin Nilsen wrote: > --- gcc/testsuite/gcc.target/powerpc/builtins-9.c (nonexistent) > +++ gcc/testsuite/gcc.target/powerpc/builtins-9.c (working copy) > @@ -0,0 +1,21 @@ > +/* { dg-do compile } */ > +/* { dg-require-effective-target

[PATCH v2, rs6000] Fix implementation of vec_pack (vector double, vector double) built-in function

2018-06-19 Thread Kelvin Nilsen
This patch fixes an error in the code generation for vec_pack (vector double, vector double). As previously implemented, this built-in function translates to the vpkudum instruction. This patch causes vec_pack (vector double, vector double) to behave the same as vec_float2 for the same type