On 6/6/18 2:54 PM, Segher Boessenkool wrote: > On Tue, Jun 05, 2018 at 10:03:46PM -0500, Peter Bergner wrote: >> On 6/5/18 3:22 PM, Segher Boessenkool wrote: >>> Ah, that's the crux. Thanks. Add a comment to the code please? >> >> Like so? I tried placing it next to the mcpu=power8 clause, but it seems >> you cannot place comments there, as it creates syntax errors during the >> build. >> >> /* Common ASM definitions used by ASM_SPEC among the various targets for >> handling -mcpu=xxx switches. There is a parallel list in >> driver-rs6000.c to >> provide the default assembler options if the user uses -mcpu=native, so >> if >> - you make changes here, make them also there. */ >> + you make changes here, make them also there. PR63177: Do not pass >> -mpower8 >> + to the assembler if -mpower9-vector was also used. */ >> #define ASM_CPU_SPEC \ >> ... > > Yes that's great, thanks! Backports are okay, too.
Looking through the release branches, I had to backport this all the way back to GCC 6. Testing on all the release branches were clean so I have committed everywhere. Thanks! Peter
