Re: [PATCH, rs6000] PR84371 Update builtins-3*.c tests for power9 codegen

2018-02-19 Thread Peter Bergner
On 2/15/18 3:56 PM, Will Schmidt wrote: > -/* { dg-options "-mcpu=power8" } */ > +/* { dg-options "-maltivec -mcpu=power8" } */ Why did you add -maltivec? That flag is enabled implicitly by -mcpu=power8 which we're already using? Peter

Re: [PATCH, rs6000] PR84371 Update builtins-3*.c tests for power9 codegen

2018-02-16 Thread Segher Boessenkool
Hi! On Thu, Feb 15, 2018 at 03:56:10PM -0600, Will Schmidt wrote: >This fixes the scan-assembler errors as seen on power9. While verifying > that fix and ensuring coverage, cleaned up some nearby bits as well. > 2018-02-15 Will Schmidt > > target/pr84371

[PATCH, rs6000] PR84371 Update builtins-3*.c tests for power9 codegen

2018-02-15 Thread Will Schmidt
Hi, This fixes the scan-assembler errors as seen on power9. While verifying that fix and ensuring coverage, cleaned up some nearby bits as well. builtins-3.c: Add/update options and skip-if stanzas, this test specifically targets vsx/power6. builtins-3-p9.c: Update stanzas to require