Szelethus marked 2 inline comments as done.
Szelethus added inline comments.


================
Comment at: test/Analysis/invalid-analyzer-config-value.c:72
+// even if -analyze isn't specified.
+// RUN: %clang --analyze -Xclang -analyzer-config -Xclang remember=TheVasa %s
+
----------------
Szelethus wrote:
> NoQ wrote:
> > This run-line *does* have `-analyze` specified, because that's what 
> > `--analyze` expands to. I guess the test that we need would be along the 
> > lines of `clang -Xclang -analyzer-config ...` without `--analyze`.
>  Uhhhh sorry about that.
I checked this against a completely different file, and it works.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55823/new/

https://reviews.llvm.org/D55823



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

Reply via email to