Re: [PATCH v3] kbuild: Set KBUILD_CFLAGS before incl. arch Makefile

From: Masahiro Yamada
Date: Wed Nov 22 2017 - 23:25:03 EST


Hi.

2017-11-18 13:09 GMT+09:00 Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>:
> 2017-11-16 5:42 GMT+09:00 Nick Desaulniers <ndesaulniers@xxxxxxxxxx>:
>> From: Chris Fries <cfries@xxxxxxxxxx>
>>
>> Set the clang KBUILD_CFLAGS up before including arch/ Makefiles,
>> so that ld-options (etc.) can work correctly.
>>
>> This fixes errors with clang such as ld-options trying to CC
>> against your host architecture, but LD trying to link against
>> your target architecture.
>>
>> Reviewed-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx>
>> Signed-off-by: Chris Fries <cfries@xxxxxxxxxx>
>> Signed-off-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
>> Suggested-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
>> Tested-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx>
>> ---
>
> Applied to linux-kbuild/kbuild. Thanks!
>
> --
> Best Regards
> Masahiro Yamada

After more thought, I picked up your v2.
(clang variables before kernel configuration)

Linus suggests to move compiler flag testing to Kconfig.
To do it, we need to feed target CC information to Kconfig.

Sorry for spending your time.


--
Best Regards
Masahiro Yamada