[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