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

[Thomas Weißschuh]

On 2026-01-12 14:58:08+0800, WangYuli wrote:
> 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.

For which value of 'LLVM' would a silent fallback be the thing to do?


Thomas