[PATCH][ARM] PR 64149: Remove -mlra/-mno-lra option for ARM.

Ramana Radhakrishnan ramana.gcc@googlemail.com
Tue Jan 20 09:17:00 GMT 2015


On Tue, Jan 13, 2015 at 1:32 PM, Matthew Wahab <matthew.wahab@arm.com> wrote:
> Hello,
>
> The LRA register alloator is enabled by default for the ARM backend and
> -mno-lra should no longer be used. This patch removes the -mlra/-mno-lra
> option from the ARM backend.
>
> arm-none-linux-gnueabihf passes gcc-check with no new failures.
>
> Matthew
>
> 2015-01-13  Matthew Wahab  <matthew.wahab@arm.com>
>
>         PR target/64149
>         * config/arm/arm.opt: Remove lra option and arm_lra_flag variable.
>         * config/arm/arm.h (MODE_BASE_REG_CLASS): Remove use of
> arm_lra_flag,
>         replace the conditional with it's true branch.
>         * config/arm/arm.c (TARGET_LRA_P): Set to hook_bool_void_true.
>         (arm_lra_p): Remove.
>         * testsuite/gcc.target/arm/thumb1-far-jump-3.c: Remove.


This is OK - Usually we don't remove command line options but given
this was always a transitional option, I'd rather folks with broken
packages with LRA actually removed the mno-lra option from their
packages and reported issues that we could fix.

An entry for changes.html please for both ARM and AArch64 backends.
Also please work on removing some of the dead reload code now from the
ARM backend too :) .


Ramana



More information about the Gcc-patches mailing list