On Fri, Jan 15, 2021 at 3:15 AM Miguel Ojeda <miguel.ojeda.sandonis@xxxxxxxxx> wrote: > > On Thu, Jan 14, 2021 at 5:25 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > > > The cc-version.sh also shows the error if the compiler is old: > > > > $ make defconfig CC=clang-9 > > *** Default configuration is based on 'x86_64_defconfig' > > *** > > *** Compiler is too old. > > *** Your Clang version: 9.0.1 > > *** Minimum Clang version: 10.0.1 > > *** > > scripts/Kconfig.include:46: Sorry, this compiler is unsupported. > > That looks nice. Hopefully we can do the same approach for other tools too! Yes, I plan to merge scripts/ld-version.sh and scripts/lld-version.sh in a similar way, and move the version check as well once the following cleanups land in the upstream: https://patchwork.kernel.org/project/linux-kbuild/patch/20201212165431.150750-1-masahiroy@xxxxxxxxxx/ https://patchwork.kernel.org/project/linux-kbuild/patch/20201212165431.150750-2-masahiroy@xxxxxxxxxx/ > > I put the stub for ICC because I see <linux/compiler-intel.h> although > > I am not sure if building the kernel with ICC is well-supported. > > I doubt it, and there seems to be no maintainer listed either. I think > it could be considered for removal in an RFC. Yes, but that would require higher level acks, and consult x86 and intel folks. Please let this patch land first, then we will discuss whether ICC is still used or not. > Cheers, > Miguel -- Best Regards Masahiro Yamada