dexonsmith added a comment.

In D107155#2916664 <https://reviews.llvm.org/D107155#2916664>, @dblaikie wrote:

> Hmm, don't we have some other substitution feature that avoids the need to 
> add the %? (like, I think LLVM's tests don't usually use the % for many tool 
> names - but they still make for good copy/pastable command lines, if I'm not 
> mistaken?)

I think that'd be adding clang-scan-deps somehow to the `tools` variable in 
clang/test/lit.cfg.py:
 https://github.com/llvm/llvm-project/blob/main/clang/test/lit.cfg.py#L59
(which will then call `add_tool_substitutions` at 
https://github.com/llvm/llvm-project/blob/main/clang/test/lit.cfg.py#L111)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107155

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to