[PATCH] D67420: [analyzer] NFC: Separate PathDiagnosticConsumer options from AnalyzerOptions.

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 15 08:01:02 PDT 2019


Szelethus added inline comments.


================
Comment at: clang/include/clang/Analysis/PathDiagnostic.h:81
+  /// because deterministic mode is always superior when done right, but
+  /// for some consumers is experimental and needs to be off by default.
+  bool ShouldWriteStableReportFilename;
----------------
gribozavr wrote:
> "but for some consumers is experimental" -- parse error. What is experimental?
I suspect you copied these from the `.def` file, did you change the descriptions there too?


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

https://reviews.llvm.org/D67420





More information about the cfe-commits mailing list