[PATCH] D92191: [clang-scan-deps] Add support for clang-cl
Sylvain Audi via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 21 10:06:26 PST 2021
saudi marked an inline comment as done.
saudi added inline comments.
================
Comment at: clang/test/ClangScanDeps/Inputs/static-analyzer-cdb.json:4-5
"directory": "DIR",
- "command": "clang --analyze DIR/static-analyzer.c",
- "file": "DIR/static-analyzer.c"
+ "command": "clang --analyze -- DIR/static-analyzer_clang.c",
+ "file": "DIR/static-analyzer_clang.c"
+},
----------------
dexonsmith wrote:
> It's not clear why you made these changes. Can you clarify?
The added "--" was not necessary, I removed it.
The added "_clang" was because I added a `CHECK:` line for the `.c` file, but "static-analyzer.c" would cause false positives with %S.
-> see the new comment in the static-analyzer.c
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92191/new/
https://reviews.llvm.org/D92191
More information about the cfe-commits
mailing list