"Moore, Catherine" <catherine_mo...@mentor.com> writes:
> Index: config/mips/mips.opt
> ===================================================================
> --- config/mips/mips.opt      (revision 199610)
> +++ config/mips/mips.opt      (working copy)
> @@ -141,6 +141,10 @@ membedded-data
>  Target Report Var(TARGET_EMBEDDED_DATA)
>  Use ROM instead of RAM
>  
> +meva
> +Target Report Var(TARGET_EVA)
> +Use microMIPS32/MIPS32 Enhanced VA instructions

This is likely to get out of date if EVA is ever extended to MIPS64.
It's probably easier just to drop the "microMIPS32/MIPS32" bit.

> Index: config/mips/mips.h
> ===================================================================
> --- config/mips/mips.h        (revision 199610)
> +++ config/mips/mips.h        (working copy)
> @@ -399,6 +399,11 @@ struct mips_cpu_info {
>        if (TARGET_MCU)                                                        
> \
>       builtin_define ("__mips_mcu");                                  \
>                                                                       \
> +      if (TARGET_EVA)                                                        
> \
> +     {                                                               \
> +       builtin_define ("__mips_eva");                                \
> +     }                                                               \
> +                                                                     \

This should be:

      if (TARGET_EVA)                                                   \
        builtin_define ("__mips_eva");                                  \

You need to document the option too.

Thanks,
Richard

Reply via email to