On Wed, 3 Mar 2021 at 14:55, Richard Earnshaw (lists)
<richard.earns...@arm.com> wrote:
>
> Hopefully this change will reduce the number of times Christophe is
> needing to tweak the testsuite.
>

Thanks!

I guess this means we can now do some cleanup in the testsuite?
Did you have a quick look at the amount of tests involved?

Christophe

> --------------
>
> Arm processors can support up to two instruction sets.  Some early
> cores only support the traditional A32 (Arm) instructions, while some
> more recent devices only support T32 (Thumb) instructions.
>
> When configuring the compiler, --with-mode can be used to select the
> default instruction set to target if the user has not made an explicit
> choice, but this can cause needless problems if the default is not
> supported by the requested CPU.
>
> To fix this this patch adjusts the way that the --with-mode selection
> is processed so that it can take into account the selected CPU or
> architecture and not create a meaningless combination.
>
> gcc:
>         * common/config/arm/arm-common.c: Include configargs.h.
>         (arm_config_default): New function.
>         (arm_target_mode): Renamed from arm_target_thumb_only.  Handle
>         processors that do not support Thumb.  Take into account the
>         --with-mode configuration setting for selecting the default.
>         * config/arm/arm.h (OPTION_DEFAULT_SPECS): Remove entry for 'mode'.
>         (TARGET_MODE_SPEC_FUNCTIONS): Update for function name change.
> ---
>  gcc/common/config/arm/arm-common.c | 49 ++++++++++++++++++++++++++----
>  gcc/config/arm/arm.h               | 10 +++---
>  2 files changed, 49 insertions(+), 10 deletions(-)
>
>

Reply via email to