[Lex] Make MacroDirective::findDirectiveAtLoc take const SourceManager

[Szelethus]

Summary

I'm currently working on including macro expansions in the Static Analyzer's plist output, where I can only access a const SourceManager.

[dblaikie]

Looks fine

[Szelethus]

Thanks! That was super fast!

include/clang/Lex/MacroInfo.h

public:
 
   const MacroInfo *getMacroInfo() const {
     return getDefinition().getMacroInfo();
   }
   MacroInfo *getMacroInfo() { return getDefinition().getMacroInfo(); }
 
   /// 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;
 
   static bool classof(const MacroDirective *) { return true; }
 };
 
 /// A directive for a defined macro or a macro imported from a module.
 class DefMacroDirective : public MacroDirective {

lib/Lex/MacroInfo.cpp

if (!isPublic.hasValue())
       isPublic = VisMD->isPublic();
   }
 
   return DefInfo(nullptr, UndefLoc,
                  !isPublic.hasValue() || isPublic.getValue());
 }
 
 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.
         SM.isBeforeInTranslationUnit(Def.getLocation(), L))
       return (!Def.isUndefined() ||
               SM.isBeforeInTranslationUnit(L, Def.getUndefLocation()))
                   ? Def : DefInfo();
   }