[PATCH] D85788: [Clang test] Update to allow passing extra default clang arguments in use_clang

Evgenii Stepanov via Phabricator via cfe-commits cfe-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 cfe-commits mailing list