Currently LLVM's integrated assembler does not recognize .w form
of the pld instructions (LLVM Bug 40972 [0]):
./arch/arm/include/asm/processor.h:133:5: error: invalid instruction
"pldw.w\t%a0 \n"
^
<inline asm>:2:1: note: instantiated into assembly here
pldw.w [r0]
^
1 error generated.
The W macro for generating wide instructions when targeting Thumb-2
is not strictly required for the preload data instructions (pld, pldw)
since they are only available as wide instructions. The GNU assembler
works with or without the .w appended when compiling an Thumb-2 kernel.
Drop the macro to work around LLVM Bug 40972 issue.
[0] https://bugs.llvm.org/show_bug.cgi?id=40972
Signed-off-by: Stefan Agner <[email protected]>
---
Changes in v2:
- Reword commit message to reflect the fact that this is a work around
for LLVM.
arch/arm/include/asm/processor.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/include/asm/processor.h b/arch/arm/include/asm/processor.h
index 57fe73ea0f72..5d06f75ffad4 100644
--- a/arch/arm/include/asm/processor.h
+++ b/arch/arm/include/asm/processor.h
@@ -135,8 +135,8 @@ static inline void prefetchw(const void *ptr)
__asm__ __volatile__(
".arch_extension mp\n"
__ALT_SMP_ASM(
- WASM(pldw) "\t%a0",
- WASM(pld) "\t%a0"
+ "pldw\t%a0",
+ "pld\t%a0"
)
:: "p" (ptr));
}
--
2.21.0
The assembler option -mauto-it is no longer a valid option. The last
remaining references have been removed from the documentation in
July 2009 [0].
The currently supported binutils version is 2.20 (released in
September 2009) or higher where gas supports -mimplicit-it=always.
Drop the fallback to -mauto-it and use -mimplicit-it=always only.
[0] https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=529707530657a333a304c651c808ea630c955223
Signed-off-by: Stefan Agner <[email protected]>
Reviewed-by: Nick Desaulniers <[email protected]>
---
Changes in v2:
- Drop $(comma) since we are no longer use the call to as-option.
arch/arm/Makefile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm/Makefile b/arch/arm/Makefile
index df3ad82d312c..eb9d38e77cf0 100644
--- a/arch/arm/Makefile
+++ b/arch/arm/Makefile
@@ -113,8 +113,7 @@ CFLAGS_ABI +=-funwind-tables
endif
ifeq ($(CONFIG_THUMB2_KERNEL),y)
-AFLAGS_AUTOIT :=$(call as-option,-Wa$(comma)-mimplicit-it=always,-Wa$(comma)-mauto-it)
-CFLAGS_ISA :=-mthumb $(AFLAGS_AUTOIT)
+CFLAGS_ISA :=-mthumb -Wa,-mimplicit-it=always
AFLAGS_ISA :=$(CFLAGS_ISA) -Wa$(comma)-mthumb
# Work around buggy relocation from gas if requested:
ifeq ($(CONFIG_THUMB2_AVOID_R_ARM_THM_JUMP11),y)
--
2.21.0
On Sun, Mar 17, 2019 at 4:05 PM Stefan Agner <[email protected]> wrote:
>
> Currently LLVM's integrated assembler does not recognize .w form
> of the pld instructions (LLVM Bug 40972 [0]):
>
> ./arch/arm/include/asm/processor.h:133:5: error: invalid instruction
> "pldw.w\t%a0 \n"
> ^
> <inline asm>:2:1: note: instantiated into assembly here
> pldw.w [r0]
> ^
> 1 error generated.
>
> The W macro for generating wide instructions when targeting Thumb-2
> is not strictly required for the preload data instructions (pld, pldw)
> since they are only available as wide instructions. The GNU assembler
> works with or without the .w appended when compiling an Thumb-2 kernel.
>
> Drop the macro to work around LLVM Bug 40972 issue.
>
> [0] https://bugs.llvm.org/show_bug.cgi?id=40972
>
> Signed-off-by: Stefan Agner <[email protected]>
Thanks for the bug report and patch.
Reviewed-by: Nick Desaulniers <[email protected]>
Just curious, there are only 3 other expansion sites of this macro.
Are any of those problematic? Looks like nop.w, sev.w and b.w?
> ---
> Changes in v2:
> - Reword commit message to reflect the fact that this is a work around
> for LLVM.
>
> arch/arm/include/asm/processor.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/include/asm/processor.h b/arch/arm/include/asm/processor.h
> index 57fe73ea0f72..5d06f75ffad4 100644
> --- a/arch/arm/include/asm/processor.h
> +++ b/arch/arm/include/asm/processor.h
> @@ -135,8 +135,8 @@ static inline void prefetchw(const void *ptr)
> __asm__ __volatile__(
> ".arch_extension mp\n"
> __ALT_SMP_ASM(
> - WASM(pldw) "\t%a0",
> - WASM(pld) "\t%a0"
> + "pldw\t%a0",
> + "pld\t%a0"
> )
> :: "p" (ptr));
> }
> --
> 2.21.0
>
--
Thanks,
~Nick Desaulniers
On 18.03.2019 19:09, Nick Desaulniers wrote:
> On Sun, Mar 17, 2019 at 4:05 PM Stefan Agner <[email protected]> wrote:
>>
>> Currently LLVM's integrated assembler does not recognize .w form
>> of the pld instructions (LLVM Bug 40972 [0]):
>>
>> ./arch/arm/include/asm/processor.h:133:5: error: invalid instruction
>> "pldw.w\t%a0 \n"
>> ^
>> <inline asm>:2:1: note: instantiated into assembly here
>> pldw.w [r0]
>> ^
>> 1 error generated.
>>
>> The W macro for generating wide instructions when targeting Thumb-2
>> is not strictly required for the preload data instructions (pld, pldw)
>> since they are only available as wide instructions. The GNU assembler
>> works with or without the .w appended when compiling an Thumb-2 kernel.
>>
>> Drop the macro to work around LLVM Bug 40972 issue.
>>
>> [0] https://bugs.llvm.org/show_bug.cgi?id=40972
>>
>> Signed-off-by: Stefan Agner <[email protected]>
>
> Thanks for the bug report and patch.
> Reviewed-by: Nick Desaulniers <[email protected]>
>
> Just curious, there are only 3 other expansion sites of this macro.
> Are any of those problematic? Looks like nop.w, sev.w and b.w?
>
All three sites are in inline assembly, and I did a bunch of successful
Thumb2 builds using the integrated assembler, so I think all those sites
have been assembled by LLVM successfully.
Also confirmed those three instructions with the reproducer example from
the LLVM bug above.
--
Stefan
>> ---
>> Changes in v2:
>> - Reword commit message to reflect the fact that this is a work around
>> for LLVM.
>>
>> arch/arm/include/asm/processor.h | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/processor.h b/arch/arm/include/asm/processor.h
>> index 57fe73ea0f72..5d06f75ffad4 100644
>> --- a/arch/arm/include/asm/processor.h
>> +++ b/arch/arm/include/asm/processor.h
>> @@ -135,8 +135,8 @@ static inline void prefetchw(const void *ptr)
>> __asm__ __volatile__(
>> ".arch_extension mp\n"
>> __ALT_SMP_ASM(
>> - WASM(pldw) "\t%a0",
>> - WASM(pld) "\t%a0"
>> + "pldw\t%a0",
>> + "pld\t%a0"
>> )
>> :: "p" (ptr));
>> }
>> --
>> 2.21.0
>>