[U-Boot] [PATCH] travis: Add sandbox/clang-7 support

Simon Glass sjg at chromium.org
Fri Oct 19 03:27:58 UTC 2018


Hi Tom,

On 9 October 2018 at 05:35, Tom Rini <trini at konsulko.com> wrote:
> On Mon, Oct 08, 2018 at 09:40:51PM -0600, Simon Glass wrote:
>> Hi Tom,
>>
>> On 7 October 2018 at 12:20, Tom Rini <trini at konsulko.com> wrote:
>> >
>> > To make testing with clang support easier, add sandbox/clang-7
>> > combination to our testing matrix.  To facilitate this, switch to using
>> > the "sources" method that the travis.yml file supports to list
>> > additional repositories and add the official one for llvm-7.  Due to
>> > buildman not supporting using clang at this time add logic to manually
>> > build a single sandbox configuration in the expected output directory so
>> > that we can still invoke all of our tests.
>> >
>> > Cc: Simon Glass <sjg at chromium.org>
>> > Signed-off-by: Tom Rini <trini at konsulko.com>
>> > ---
>> >  .travis.yml | 27 ++++++++++++++++++++-------
>> >  1 file changed, 20 insertions(+), 7 deletions(-)
>>
>> Reviewed-by: Simon Glass <sjg at chromium.org>
>>
>> What prevents buildman supporting clang?
>
> Well, I couldn't convince buildman to accept "/usr/bin/clang-7" as the
> full and entire path to the toolchain to use.  Ideas?  Thanks!

No, sadly it looks like we need to add special support for clang - as
per README.clang. We need to set CC and CROSS_COMPILE, at least.

Regards,
Simon


More information about the U-Boot mailing list