[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
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 cfe-commits mailing list