From: Kristof Umann Date: Wed, 31 Oct 2018 17:19:20 +0000 (+0000) Subject: [Lex] Make MacroDirective::findDirectiveAtLoc take const SourceManager X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=20c21b7b6835ec96fbe9810c3e51b4777818d616;p=clang [Lex] Make MacroDirective::findDirectiveAtLoc take const SourceManager I'm currently working on including macro expansions in the Static Analyzer's plist output, where I can only access a const SourceManager. Differential Revision: https://reviews.llvm.org/D53940 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345741 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Lex/MacroInfo.h b/include/clang/Lex/MacroInfo.h index 0cc1cb92e6..a06de132b4 100644 --- a/include/clang/Lex/MacroInfo.h +++ b/include/clang/Lex/MacroInfo.h @@ -395,7 +395,8 @@ public: /// Find macro definition active in the specified source location. If /// this macro was not defined there, return NULL. - const DefInfo findDirectiveAtLoc(SourceLocation L, SourceManager &SM) const; + const DefInfo findDirectiveAtLoc(SourceLocation L, + const SourceManager &SM) const; void dump() const; diff --git a/lib/Lex/MacroInfo.cpp b/lib/Lex/MacroInfo.cpp index 4ed69ecc46..434c120075 100644 --- a/lib/Lex/MacroInfo.cpp +++ b/lib/Lex/MacroInfo.cpp @@ -200,7 +200,8 @@ MacroDirective::DefInfo MacroDirective::getDefinition() { } const MacroDirective::DefInfo -MacroDirective::findDirectiveAtLoc(SourceLocation L, SourceManager &SM) const { +MacroDirective::findDirectiveAtLoc(SourceLocation L, + const SourceManager &SM) const { assert(L.isValid() && "SourceLocation is invalid."); for (DefInfo Def = getDefinition(); Def; Def = Def.getPreviousDefinition()) { if (Def.getLocation().isInvalid() || // For macros defined on the command line.