[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