[PATCH] buildman: Correct override toolchain support for clang

[Tom Rini]

On Fri, Nov 07, 2025 at 01:23:00PM +0100, Simon Glass wrote:
> On Fri, 7 Nov 2025 at 00:28, Tom Rini <trini at konsulko.com> wrote:
> >
> > When initially adding support for building with clang with buildman,
> > there was some minimum logic in the Makefile to inform clang what the
> > expected target was, without CROSS_COMPILE being set. This has long been
> > deprecated / removed from the Linux kernel, and also not functional in
> > U-Boot. The normal expectation, especially so that an intentional
> > mix-and-match of GNU binutils and LLVM C compiler can be used relies on
> > CROSS_COMPILE also being set and passed. To have buildman do this we
> > only need to remove the logic that was not setting CROSS_COMPILE when
> > we had an override. Correct the comment about what override_toolchain
> > does as it is functional outside of sandbox too.
> >
> > Signed-off-by: Tom Rini <trini at konsulko.com>
> > ---
> > Cc: Simon Glass <sjg at chromium.org>
> >
> >  tools/buildman/toolchain.py | 7 ++-----
> >  1 file changed, 2 insertions(+), 5 deletions(-)
> >
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> I don't recall trying clang with real boards.

I did for a few years until things broke and I didn't have time nor
incentive to try and fix. Now that someone has fixed clang builds in
general and I've gotten someone to work on labgrid with upstream, we
should have a lab posted shortly and clang will be part of the set of
builds that gets tested.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20251107/21c2d610/attachment.sig>