[clang] 7b6bf8b - [NFC][analyzer] const ptr param in AnalysisConsumer::getModeForDecl (#170145)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 1 06:49:03 PST 2025
Author: Arseniy Zaostrovnykh
Date: 2025-12-01T14:48:59Z
New Revision: 7b6bf8b060f74669d7027d33f488a35cfb448b29
URL: https://github.com/llvm/llvm-project/commit/7b6bf8b060f74669d7027d33f488a35cfb448b29
DIFF: https://github.com/llvm/llvm-project/commit/7b6bf8b060f74669d7027d33f488a35cfb448b29.diff
LOG: [NFC][analyzer] const ptr param in AnalysisConsumer::getModeForDecl (#170145)
This is a tiny change that would make the function contract more clear
and our work downstream easier.
Added:
Modified:
clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
Removed:
################################################################################
diff --git a/clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp b/clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
index e0deec16afb12..7b5dca683d5a7 100644
--- a/clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
+++ b/clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
@@ -364,7 +364,7 @@ class AnalysisConsumer : public AnalysisASTConsumer,
void storeTopLevelDecls(DeclGroupRef DG);
/// Check if we should skip (not analyze) the given function.
- AnalysisMode getModeForDecl(Decl *D, AnalysisMode Mode);
+ AnalysisMode getModeForDecl(const Decl *D, AnalysisMode Mode);
void runAnalysisOnTranslationUnit(ASTContext &C);
/// Print \p S to stderr if \c Opts.AnalyzerDisplayProgress is set.
@@ -677,7 +677,7 @@ void AnalysisConsumer::HandleTranslationUnit(ASTContext &C) {
}
AnalysisConsumer::AnalysisMode
-AnalysisConsumer::getModeForDecl(Decl *D, AnalysisMode Mode) {
+AnalysisConsumer::getModeForDecl(const Decl *D, AnalysisMode Mode) {
if (!Opts.AnalyzeSpecificFunction.empty() &&
AnalysisDeclContext::getFunctionName(D) != Opts.AnalyzeSpecificFunction &&
cross_tu::CrossTranslationUnitContext::getLookupName(D).value_or("") !=
@@ -695,7 +695,7 @@ AnalysisConsumer::getModeForDecl(Decl *D, AnalysisMode Mode) {
const SourceManager &SM = Ctx->getSourceManager();
- const SourceLocation Loc = [&SM](Decl *D) -> SourceLocation {
+ const SourceLocation Loc = [&SM](const Decl *D) -> SourceLocation {
const Stmt *Body = D->getBody();
SourceLocation SL = Body ? Body->getBeginLoc() : D->getLocation();
return SM.getExpansionLoc(SL);
More information about the cfe-commits
mailing list