Re: [PATCH] kbuild: Reject unexpected values for LLVM=

[WangYuli]

Hi Thomas Weißschuh,

On 2026/1/12 14:43, Thomas Weißschuh wrote:

> The LLVM argument is documented to accept one of three forms:
> * a literal '1' to use the default 'clang',
> * a toolchain prefix path, ending in a trailing '/',
> * a version suffix.
>
> All other values are silently treated as '1'. If for example
> the user accidentally forgets the trailing '/' of a toolchain prefix,
> kbuild will unexpectedly and silently fall back to the system toolchain.
>
> Instead report an error if the user specified an invalid value for LLVM.
"Falling back to the system toolchain" may be an acceptable behavior; I 
think reporting a warning here rather than an error would be more 
appropriate.

Thanks,

--

WangYuli