[PATCH] D53296: [analyzer] New flag to print all -analyzer-config options

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 29 16:27:49 PDT 2018


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

Thanks! :)



================
Comment at: lib/FrontendTool/ExecuteCompilerInvocation.cpp:255
+    ento::printAnalyzerConfigList(llvm::outs());
+    return true;
+  }
----------------
george.karpenkov wrote:
> Hm, should we return here?
> If there are errors, which are only printed in the next part, should they still be printed?
I'd prefer to return as every other option. I think the user, if `-analyzer-config-help` was supplied, is only interested in a config option list.


https://reviews.llvm.org/D53296





More information about the cfe-commits mailing list