Re: [PATCH 1/3] ARM: use arch_extension directive instead of arch argument
From: Krzysztof Kozlowski
Date: Mon Mar 25 2019 - 06:53:50 EST
On Sat, 23 Mar 2019 at 16:52, Stefan Agner <stefan@xxxxxxxx> wrote:
> The LLVM Target parser currently does not allow to specify the security
> extension as part of -march (see also LLVM Bug 40186 ). When trying
> to use Clang with LLVM's integrated assembler, this leads to a build
> errors such as this:
> clang-8: error: the clang compiler does not support '-Wa,-march=armv7-a+sec'
> Use ".arch_extension sec" to enable the security extension in a more
> portable fasion.
> Note that this is technically not exactly the same as the old code
> checked for availabilty of the security extension by calling as-instr.
> However, there are already other sites which use ".arch_extension sec"
> unconditionally, hence de-facto we need an assembler capable of
> ".arch_extension sec" already today (arch/arm/mm/proc-v7.S). The
> arch extension "sec" is available since binutils 2.21 according to
> its documentation .
>  https://bugs.llvm.org/show_bug.cgi?id=40186
>  https://sourceware.org/binutils/docs-2.21/as/ARM-Options.html
> Signed-off-by: Stefan Agner <stefan@xxxxxxxx>
> arch/arm/mach-bcm/Makefile | 3 ---
> arch/arm/mach-bcm/bcm_kona_smc.c | 2 --
> arch/arm/mach-exynos/Makefile | 4 ----
> arch/arm/mach-exynos/exynos-smc.S | 2 +-
> arch/arm/mach-exynos/sleep.S | 2 +-
Acked-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>