On Fri, Mar 07, 2025 at 08:29:25PM -0800, Kees Cook wrote: > The use of -ffreestanding is a leftover that is only needed for certain > versions of Clang. Adjust this to be Clang-only. A later patch will make > this a versioned check. > > Signed-off-by: Kees Cook <k...@kernel.org>
This could also adjust the comment but it is probably not that big of a deal since the "temporary" is already pretty stale. Reviewed-by: Nathan Chancellor <nat...@kernel.org> > --- > Cc: Nathan Chancellor <nat...@kernel.org> > Cc: Nick Desaulniers <nick.desaulniers+l...@gmail.com> > Cc: Thomas Gleixner <t...@linutronix.de> > Cc: Ingo Molnar <mi...@redhat.com> > Cc: Borislav Petkov <b...@alien8.de> > Cc: Dave Hansen <dave.han...@linux.intel.com> > Cc: x...@kernel.org > Cc: "H. Peter Anvin" <h...@zytor.com> > --- > arch/x86/Makefile | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/x86/Makefile b/arch/x86/Makefile > index 5b773b34768d..d25ed25fb7d9 100644 > --- a/arch/x86/Makefile > +++ b/arch/x86/Makefile > @@ -137,8 +137,10 @@ ifeq ($(CONFIG_X86_32),y) > include $(srctree)/arch/x86/Makefile_32.cpu > KBUILD_CFLAGS += $(cflags-y) > > + ifeq ($(CONFIG_CC_IS_CLANG),y) > # temporary until string.h is fixed > KBUILD_CFLAGS += -ffreestanding > + endif > > ifeq ($(CONFIG_STACKPROTECTOR),y) > ifeq ($(CONFIG_SMP),y) > -- > 2.34.1 >
