Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-12 Thread Arnd Bergmann
On Thu, Nov 12, 2020 at 9:42 AM Ard Biesheuvel wrote: > Although I must admit I am still on the fence when it comes to the > policy around rewriting perfectly valid code like this to accommodate > a toolchain that nobody is likely to use to build the code in > question. > > Perhaps we should at

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-12 Thread Ard Biesheuvel
On Sat, 7 Nov 2020 at 01:11, Jian Cai wrote: > > This patch replaces 6 IWMMXT instructions Clang's integrated assembler > does not support in iwmmxt.S using macros, while making sure GNU > assembler still emit the same instructions. This should be easier than > providing full IWMMXT support in

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-10 Thread Ard Biesheuvel
On Tue, 10 Nov 2020 at 21:38, Arnd Bergmann wrote: > > On Tue, Nov 10, 2020 at 9:11 PM 'Nick Desaulniers' via Clang Built > Linux wrote: > > > > On Tue, Nov 10, 2020 at 12:10 PM Jian Cai wrote: > > > > > > I tried to verify with ixp4xx_defconfig, and I noticed it also used > > >

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-10 Thread Arnd Bergmann
On Tue, Nov 10, 2020 at 9:11 PM 'Nick Desaulniers' via Clang Built Linux wrote: > > On Tue, Nov 10, 2020 at 12:10 PM Jian Cai wrote: > > > > I tried to verify with ixp4xx_defconfig, and I noticed it also used > > CONFIG_CPU_BIG_ENDIAN=y to enable big endianness as follows, > > > > linux$ grep

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-10 Thread Nick Desaulniers
On Tue, Nov 10, 2020 at 12:10 PM Jian Cai wrote: > > I tried to verify with ixp4xx_defconfig, and I noticed it also used > CONFIG_CPU_BIG_ENDIAN=y to enable big endianness as follows, > > linux$ grep ENDIAN arch/arm/configs/ixp4xx_defconfig > CONFIG_CPU_BIG_ENDIAN=y > > Also it appeared

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-10 Thread Nick Desaulniers
On Mon, Nov 9, 2020 at 11:36 PM Ard Biesheuvel wrote: > > BE32 != BE8 Oh? Sorry, what does BE8 stand for? arch/arm/mm/Kconfig says: CONFIG_CPU_ENDIAN_BE8 Support for the BE-8 (big-endian) mode on ARMv6 and ARMv7 processors. vs: CPU_ENDIAN_BE32 Support for the BE-32 (big-endian) mode on

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-09 Thread Ard Biesheuvel
On Tue, 10 Nov 2020 at 02:55, Nick Desaulniers wrote: > > On Sat, Nov 7, 2020 at 12:29 AM Ard Biesheuvel wrote: > > > > On Sat, 7 Nov 2020 at 01:11, Jian Cai wrote: > > > > > > This patch replaces 6 IWMMXT instructions Clang's integrated assembler > > > does not support in iwmmxt.S using

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-09 Thread Nick Desaulniers
On Sat, Nov 7, 2020 at 12:29 AM Ard Biesheuvel wrote: > > On Sat, 7 Nov 2020 at 01:11, Jian Cai wrote: > > > > This patch replaces 6 IWMMXT instructions Clang's integrated assembler > > does not support in iwmmxt.S using macros, while making sure GNU > > assembler still emit the same

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-07 Thread Ard Biesheuvel
On Sat, 7 Nov 2020 at 01:11, Jian Cai wrote: > > This patch replaces 6 IWMMXT instructions Clang's integrated assembler > does not support in iwmmxt.S using macros, while making sure GNU > assembler still emit the same instructions. This should be easier than > providing full IWMMXT support in

[PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-06 Thread Jian Cai
This patch replaces 6 IWMMXT instructions Clang's integrated assembler does not support in iwmmxt.S using macros, while making sure GNU assembler still emit the same instructions. This should be easier than providing full IWMMXT support in Clang. "Intel Wireless MMX Technology - Developer Guide -