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.

> @@ -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))

?

  Maciej

Reply via email to