[PATCH] D127486: [analyzer][NFC] Inline AnalyzerOptions::getUserMode()

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 13 12:17:34 PDT 2022


steakhal added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.h:362-364
-  /// Retrieves and sets the UserMode. This is a high-level option,
-  /// which is used to set other low-level options. It is not accessible
-  /// outside of AnalyzerOptions.
----------------
martong wrote:
> I would miss this comment.
Which part do you miss?

The `AnalyzerOptions.def` already has a //half descent// description about this:
> (string) Controls the high-level analyzer mode, which influences the default settings for some of the lower-level config options (such as IPAMode). Value: "deep", "shallow".

If you insist, let me know what comment should I put and where.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127486



More information about the cfe-commits mailing list