[PATCH] D103154: [lit][test] Improve testing of use_llvm_tool

Thu May 27 00:45:26 PDT 2021

jhenderson added a comment.

In D103154#2783705 <https://reviews.llvm.org/D103154#2783705>, @MaskRay wrote:

> I just understand what D102630 <https://reviews.llvm.org/D102630> and this do. Now `rm $build/bin/ld.lld; llvm-lit something.s` will not pick a `ld.lld` from PATH. Looks great!
>
>   curl -L 'https://reviews.llvm.org/D103154?download=1' | patch -p1
>   ninja check-lit
>
> says use-llvm-tool.py fails.

I stuffed up the diff somehow. The changes in config.py should be part of D103156 <https://reviews.llvm.org/D103156>, not here. I suspect that's what's breaking the test. Will investigate and fix.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103154/new/

https://reviews.llvm.org/D103154