Messages in this thread | | | Date | Tue, 25 Feb 2014 17:32:53 -0800 | From | "H. Peter Anvin" <> | Subject | Re: [PATCH 5/5] x86 kbuild: LLVMLinux: More cc-options added for clang |
| |
On 02/25/2014 05:08 PM, behanw@converseincode.com wrote: > > +# enforce no-sse for clang > +ifneq ($(COMPILER),clang) > + KBUILD_CFLAGS += $(call cc-option,-mno-sse) > + KBUILD_CFLAGS += $(call cc-option,-mpreferred-stack-boundary=3) > +endif > +
I'm *very* confused. You're doing ifneq here but you're talking about it as if you are *adding* them for Clang, also these options are already added elsewhere (lines 57, 64, 86, 89) so why add them here?
-hpa
| |