[PATCH] D85788: [Clang test] Update to allow passing extra default clang arguments in use_clang
Evgenii Stepanov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 21 13:32:50 PST 2020
eugenis added a comment.
In D85788#2449299 <https://reviews.llvm.org/D85788#2449299>, @aqjune wrote:
> In D85788#2444136 <https://reviews.llvm.org/D85788#2444136>, @guiand wrote:
>
>> IMO it's better to just one-and-done programatically add `-Xclang -disable-noundef-analysis` to all the tests' RUN lines. That way there are no hidden flags.
>
> If a script for this can be written and people who are working for the downstream project are happy with the script, this is the best approach IMO. It is clear and explicit.
Yeah, I also think this would be the best.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85788/new/
https://reviews.llvm.org/D85788
More information about the llvm-commits
mailing list