Re: [PATCH] powerpc/32: Avoid unsupported flags with clang

2018-11-05 Thread Segher Boessenkool
On Mon, Nov 05, 2018 at 02:48:49PM +1100, Michael Ellerman wrote: > Joel Stanley writes: > > When building for ppc32 with clang these flags are unsupported: > > > > -ffixed-r2 and -mmultiple > > -CFLAGS-$(CONFIG_PPC32) := -ffixed-r2 $(MULTIPLEWORD) > > +ifndef CONFIG_CC_IS_CLANG > >

Re: [PATCH] powerpc/32: Avoid unsupported flags with clang

2018-11-04 Thread Joel Stanley
On Mon, 5 Nov 2018 at 14:18, Michael Ellerman wrote: > > I considered putting these behind a cc-option test. If someone has an > > opinion on that I'm happy to change. > > -CFLAGS-$(CONFIG_PPC32) := -ffixed-r2 $(MULTIPLEWORD) > > +ifndef CONFIG_CC_IS_CLANG > > +#CFLAGS-$(CONFIG_PPC32)

Re: [PATCH] powerpc/32: Avoid unsupported flags with clang

2018-11-04 Thread Michael Ellerman
Joel Stanley writes: > When building for ppc32 with clang these flags are unsupported: > > -ffixed-r2 and -mmultiple > > llvm's lib/Target/PowerPC/PPCRegisterInfo.cpp marks r2 as reserved on > when building for SVR4ABI and !ppc64: > > // The SVR4 ABI reserves r2 and r13 > if

Re: [PATCH] powerpc/32: Avoid unsupported flags with clang

2018-11-01 Thread Joel Stanley
On Fri, 2 Nov 2018 at 15:09, Joel Stanley wrote: > > When building for ppc32 with clang these flags are unsupported: > > -ffixed-r2 and -mmultiple > > llvm's lib/Target/PowerPC/PPCRegisterInfo.cpp marks r2 as reserved on > when building for SVR4ABI and !ppc64: > > // The SVR4 ABI reserves r2

[PATCH] powerpc/32: Avoid unsupported flags with clang

2018-11-01 Thread Joel Stanley
When building for ppc32 with clang these flags are unsupported: -ffixed-r2 and -mmultiple llvm's lib/Target/PowerPC/PPCRegisterInfo.cpp marks r2 as reserved on when building for SVR4ABI and !ppc64: // The SVR4 ABI reserves r2 and r13 if (Subtarget.isSVR4ABI()) { // We only reserve r2