Hello Richard,

On Fri, 2025-05-02 at 12:12 +0200, Richard Biener wrote:
> This flips the default to LRA for targets with an -mlra option not
> using Mask(..).
> 
>       * config/avr/avr.opt (mlra): Flip to default on.
>       * config/m68k/m68k.opt (mlra): Likewise.
>       * config/pa/pa.opt (mlra): Likewise.
>       * config/sh/sh.opt (mlra): Likewise.
> ---
>  gcc/config/avr/avr.opt   | 2 +-
>  gcc/config/m68k/m68k.opt | 2 +-
>  gcc/config/pa/pa.opt     | 2 +-
>  gcc/config/sh/sh.opt     | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/gcc/config/avr/avr.opt b/gcc/config/avr/avr.opt
> index fcd2bf68f2a..3aa4955361b 100644
> --- a/gcc/config/avr/avr.opt
> +++ b/gcc/config/avr/avr.opt
> @@ -19,7 +19,7 @@
>  ; <http://www.gnu.org/licenses/>.
>  
>  mlra
> -Target Var(avropt_lra_p) UInteger Init(0) Optimization Undocumented
> +Target Var(avropt_lra_p) UInteger Init(1) Optimization Undocumented
>  Usa LRA for reload instead of the old reload framework.  This option is 
> experimental, and it may be removed in future versions of the compiler.
>  
>  mcall-prologues
> diff --git a/gcc/config/m68k/m68k.opt b/gcc/config/m68k/m68k.opt
> index 35f86ba11ff..b664425c5a1 100644
> --- a/gcc/config/m68k/m68k.opt
> +++ b/gcc/config/m68k/m68k.opt
> @@ -147,7 +147,7 @@ Target RejectNegative Mask(LONG_JUMP_TABLE_OFFSETS)
>  Use 32-bit offsets in jump tables rather than 16-bit offsets.
>  
>  mlra
> -Target Var(m68k_lra_p) Undocumented
> +Target Var(m68k_lra_p) Init(1) Undocumented
>  Usa LRA for reload instead of the old reload framework.  This option is
>  experimental, and it may be removed in future versions of the compiler.

I just applied this patch against master and did a full native bootstrap with
all languages except Ada, Go, Rust and Cobol on Debian unstable m68k which 
worked
without any problems. Full build log in [1].

I therefore suggest to flip the switch for m68k now with reference to PR113939 
[2]
so that we can close this bug and tick off m68k on the list of targets to be 
switched
to LRA by default.

Thanks,
Adrian

> [1] 
> http://people.debian.org/~glaubitz/gcc16-m68k-lra-all-but-go-and-ada.log.gz
> [2] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113939

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Reply via email to