[PATCH, rs6000] Clean up the option_mask defines (part 3) Hi,
Per code review, the MASK_REGNAMES, OPTION_MASK_REGNAMES, MASK_PROTOTYPE, OPTION_MASK_PROTOTYPE options are not used elsewhere in the codebase. Thus it should be safe to remove them. This includes an update to a nearby comment to hint that most of the MASK_<xxx> options have now been replaced with their OPTION_MASK_<xxx> equivalents. Regtested OK on power10. OK for trunk? diff --git a/gcc/config/rs6000/rs6000.h b/gcc/config/rs6000/rs6000.h index dcf632c1f1ad..fe77a343d2e1 100644 --- a/gcc/config/rs6000/rs6000.h +++ b/gcc/config/rs6000/rs6000.h @@ -503,12 +503,13 @@ extern int rs6000_vector_align[]; answers if the arguments are not in the normal range. */ #define TARGET_MINMAX (TARGET_HARD_FLOAT && TARGET_PPC_GFXOPT \ && (TARGET_P9_MINMAX || !flag_trapping_math)) /* In switching from using target_flags to using rs6000_isa_flags, the options - machinery creates OPTION_MASK_<xxx> instead of MASK_<xxx>. For now map - OPTION_MASK_<xxx> back into MASK_<xxx>. */ + machinery creates OPTION_MASK_<xxx> instead of MASK_<xxx>. The MASK_<xxx> + options that have not yet been replaced by their OPTION_MASK_<xxxx> + equivalents are defined here. */ #define MASK_STRICT_ALIGN OPTION_MASK_STRICT_ALIGN #ifndef IN_LIBGCC2 #define MASK_POWERPC64 OPTION_MASK_POWERPC64 #endif @@ -519,18 +520,10 @@ extern int rs6000_vector_align[]; #ifdef TARGET_LITTLE_ENDIAN #define MASK_LITTLE_ENDIAN OPTION_MASK_LITTLE_ENDIAN #endif -#ifdef TARGET_REGNAMES -#define MASK_REGNAMES OPTION_MASK_REGNAMES -#endif - -#ifdef TARGET_PROTOTYPE -#define MASK_PROTOTYPE OPTION_MASK_PROTOTYPE -#endif - #ifdef TARGET_MODULO #define RS6000_BTM_MODULO OPTION_MASK_MODULO #endif