[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