https://gcc.gnu.org/g:d3b4ba120ce3b743838c3545a24554989955722a
commit r15-659-gd3b4ba120ce3b743838c3545a24554989955722a Author: YunQiang Su <s...@gcc.gnu.org> Date: Thu May 16 02:30:50 2024 +0800 MIPS: Remove -m(no-)lra option PR target/113955 The `-mlra` option was introduced in 2014 for MIPS, and was set to default since then. It's time for us to drop no-lra support by dropping -m(no-)lra options. gcc: * config/mips/mips.cc(mips_option_override): Drop mips_lra_flag variable; (mips_lra_p): Removed. (TARGET_LRA_P): Remove definition here to use the default one. * config/mips/mips.md(*mul_acc_si, *mul_acc_si_r3900, *mul_sub_si): Drop mips_lra_flag variable. * config/mips/mips.opt(-mlra): Removed. * config/mips/mips.opt.urls(mlra): Removed. Diff: --- gcc/config/mips/mips.cc | 12 ------------ gcc/config/mips/mips.md | 24 +++--------------------- gcc/config/mips/mips.opt | 4 ---- gcc/config/mips/mips.opt.urls | 2 -- 4 files changed, 3 insertions(+), 39 deletions(-) diff --git a/gcc/config/mips/mips.cc b/gcc/config/mips/mips.cc index ce764a5cb359..b63d40a357b7 100644 --- a/gcc/config/mips/mips.cc +++ b/gcc/config/mips/mips.cc @@ -20391,8 +20391,6 @@ mips_option_override (void) error ("unsupported combination: %s", "-mfp64 -mfpxx"); else if (ISA_MIPS1 && !TARGET_FLOAT32) error ("%<-march=%s%> requires %<-mfp32%>", mips_arch_info->name); - else if (TARGET_FLOATXX && !mips_lra_flag) - error ("%<-mfpxx%> requires %<-mlra%>"); /* End of code shared with GAS. */ @@ -22871,14 +22869,6 @@ mips_spill_class (reg_class_t rclass ATTRIBUTE_UNUSED, return NO_REGS; } -/* Implement TARGET_LRA_P. */ - -static bool -mips_lra_p (void) -{ - return mips_lra_flag; -} - /* Implement TARGET_IRA_CHANGE_PSEUDO_ALLOCNO_CLASS. */ static reg_class_t @@ -23307,8 +23297,6 @@ mips_bit_clear_p (enum machine_mode mode, unsigned HOST_WIDE_INT m) #undef TARGET_SPILL_CLASS #define TARGET_SPILL_CLASS mips_spill_class -#undef TARGET_LRA_P -#define TARGET_LRA_P mips_lra_p #undef TARGET_IRA_CHANGE_PSEUDO_ALLOCNO_CLASS #define TARGET_IRA_CHANGE_PSEUDO_ALLOCNO_CLASS mips_ira_change_pseudo_allocno_class diff --git a/gcc/config/mips/mips.md b/gcc/config/mips/mips.md index 26f758c90ddf..7de85123e7c2 100644 --- a/gcc/config/mips/mips.md +++ b/gcc/config/mips/mips.md @@ -1781,13 +1781,7 @@ (set_attr "mode" "SI") (set_attr "insn_count" "1,1,2") (set (attr "enabled") - (cond [(and (eq_attr "alternative" "0") - (match_test "!mips_lra_flag")) - (const_string "yes") - (and (eq_attr "alternative" "1") - (match_test "mips_lra_flag")) - (const_string "yes") - (eq_attr "alternative" "2") + (cond [(eq_attr "alternative" "1,2") (const_string "yes")] (const_string "no")))]) @@ -1811,13 +1805,7 @@ (set_attr "mode" "SI") (set_attr "insn_count" "1,1,1,2") (set (attr "enabled") - (cond [(and (eq_attr "alternative" "0") - (match_test "!mips_lra_flag")) - (const_string "yes") - (and (eq_attr "alternative" "1") - (match_test "mips_lra_flag")) - (const_string "yes") - (eq_attr "alternative" "2,3") + (cond [(eq_attr "alternative" "1,2,3") (const_string "yes")] (const_string "no")))]) @@ -2039,13 +2027,7 @@ (set_attr "mode" "SI") (set_attr "insn_count" "1,1,2") (set (attr "enabled") - (cond [(and (eq_attr "alternative" "0") - (match_test "!mips_lra_flag")) - (const_string "yes") - (and (eq_attr "alternative" "1") - (match_test "mips_lra_flag")) - (const_string "yes") - (eq_attr "alternative" "2") + (cond [(eq_attr "alternative" "1,2") (const_string "yes")] (const_string "no")))]) diff --git a/gcc/config/mips/mips.opt b/gcc/config/mips/mips.opt index c1abb36212f9..99fe93019004 100644 --- a/gcc/config/mips/mips.opt +++ b/gcc/config/mips/mips.opt @@ -413,10 +413,6 @@ msynci Target Mask(SYNCI) Use synci instruction to invalidate i-cache. -mlra -Target Var(mips_lra_flag) Init(1) Save -Use LRA instead of reload. - mlxc1-sxc1 Target Var(mips_lxc1_sxc1) Init(1) Use lwxc1/swxc1/ldxc1/sdxc1 instructions where applicable. diff --git a/gcc/config/mips/mips.opt.urls b/gcc/config/mips/mips.opt.urls index 9d166646d65c..5921d6929b28 100644 --- a/gcc/config/mips/mips.opt.urls +++ b/gcc/config/mips/mips.opt.urls @@ -222,8 +222,6 @@ UrlSuffix(gcc/MIPS-Options.html#index-msym32) msynci UrlSuffix(gcc/MIPS-Options.html#index-msynci) -; skipping UrlSuffix for 'mlra' due to finding no URLs - mlxc1-sxc1 UrlSuffix(gcc/MIPS-Options.html#index-mlxc1-sxc1)