Re: [Patch, rs6000] Require Power 8 for vec_float2 builtin

2018-02-16 Thread Carl Love
On Fri, 2018-02-16 at 10:21 -0800, Carl Love wrote: > > > > > Do you have a new testcase, too?  Or is this all covered by > > existing > > tests (which?) > > The issue was found when compiling the existing test case file with > -mcpu=power7.  No additional tests are needed for this issue. >

Re: [Patch, rs6000] Require Power 8 for vec_float2 builtin

2018-02-16 Thread Carl Love
On Fri, 2018-02-16 at 11:56 -0600, Segher Boessenkool wrote: > Hi! Segher: Per our discussion on IRC. I created the bugzilla: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422 for this issue so it is visible to people outside IBM. I have included this test file int the above bugzilla along

Re: [Patch, rs6000] Require Power 8 for vec_float2 builtin

2018-02-16 Thread Segher Boessenkool
Hi! On Fri, Feb 16, 2018 at 08:58:49AM -0800, Carl Love wrote: > This patch changes the expansion macros to the equivalent Power 8 > macros. The test now cleanly exits with the message > > error: builtin function ‘__builtin_vsx_float2_v2di’ requires the ‘- > mpower8-vector’ option > >

[Patch, rs6000] Require Power 8 for vec_float2 builtin

2018-02-16 Thread Carl Love
GCC maintainers: The following patch fixes a GCC internal compiler error. Several of the builtins use the wrong enable conditions as mentioned in Bill's issue data base (https://github.ibm.com/wschmidt/power-gcc/issue s/282). The implementation for the vec_float2() builtin, as implemented, uses