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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to