[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
Fri Oct 16 13:14:48 PDT 2020
eugenis added a comment.
I don't like the %clang_bin substitution - imho it's unclear for the test authors when to use it instead of %clang, but I can't come up with a better idea.
Is it true that %clang_bin is only necessary when the automatically added -disable-noundef-analysis flag triggers an unused argument warning, or are there other reasons?
I wonder it can be avoided by
- disable noundef analysis by default in cc1
- always add -enable-noundef-analysis in the driver when invoking cc1
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