Maciej Rozycki <maciej.rozy...@imgtec.com> writes: > On Wed, 4 May 2016, Matthew Fortune wrote: > > > diff --git a/gcc/config/mips/mips-cpus.def > > b/gcc/config/mips/mips-cpus.def index 17034f2..5df9807 100644 > > --- a/gcc/config/mips/mips-cpus.def > > +++ b/gcc/config/mips/mips-cpus.def > > @@ -44,10 +44,7 @@ MIPS_CPU ("mips4", PROCESSOR_R10000, 4, 0) > > isn't tuned to a specific processor. */ MIPS_CPU ("mips32", > > PROCESSOR_4KC, 32, PTF_AVOID_BRANCHLIKELY) MIPS_CPU ("mips32r2", > > PROCESSOR_74KF2_1, 33, PTF_AVOID_BRANCHLIKELY) > > -/* mips32r3 is micromips hense why it uses the M4K processor. > > - mips32r5 should use the p5600 processor, but there is no > definition > > - for this yet, so in the short term we will use the same processor > entry > > - as mips32r2. */ > > +/* mips32r3 is micromips hense why it uses the M4K processor. */ > > Typo here -> s/hense/hence/ -- since you've reworked the comment and > changed the line in the course anyway, you may have well taken the > opportunity and fixed it.
I saw it but chose not to fix on the basis of one change leads to another. > > @@ -150,7 +147,8 @@ MIPS_CPU ("1004kf1_1", PROCESSOR_24KF1_1, 33, 0) > > MIPS_CPU ("interaptiv", PROCESSOR_24KF2_1, 33, 0) > > > > /* MIPS32 Release 5 processors. */ > > -MIPS_CPU ("p5600", PROCESSOR_P5600, 36, PTF_AVOID_BRANCHLIKELY) > > +MIPS_CPU ("p5600", PROCESSOR_P5600, 36, PTF_AVOID_BRANCHLIKELY > > + | PTF_AVOID_IMADD) > > Not: > > MIPS_CPU ("p5600", PROCESSOR_P5600, 36, (PTF_AVOID_BRANCHLIKELY > | PTF_AVOID_IMADD)) Good spot, brackets added. I almost didn't split the line for fear of breaking the awk parsing. Thanks, Matthew diff --git a/gcc/config/mips/mips-cpus.def b/gcc/config/mips/mips-cpus.def index 5df9807..b46c86f 100644 --- a/gcc/config/mips/mips-cpus.def +++ b/gcc/config/mips/mips-cpus.def @@ -147,8 +147,8 @@ MIPS_CPU ("1004kf1_1", PROCESSOR_24KF1_1, 33, 0) MIPS_CPU ("interaptiv", PROCESSOR_24KF2_1, 33, 0) /* MIPS32 Release 5 processors. */ -MIPS_CPU ("p5600", PROCESSOR_P5600, 36, PTF_AVOID_BRANCHLIKELY - | PTF_AVOID_IMADD) +MIPS_CPU ("p5600", PROCESSOR_P5600, 36, (PTF_AVOID_BRANCHLIKELY + | PTF_AVOID_IMADD)) MIPS_CPU ("m5100", PROCESSOR_M5100, 36, PTF_AVOID_BRANCHLIKELY) MIPS_CPU ("m5101", PROCESSOR_M5100, 36, PTF_AVOID_BRANCHLIKELY) -- 2.2.1