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