From 21a7a5e8cffe48b823d8930380cb2748193e5cc5 Mon Sep 17 00:00:00 2001 From: Manuel Klimek Date: Fri, 20 Apr 2012 14:07:01 +0000 Subject: [PATCH] No need to put the SourceManager in with the ASTContext, as the ASTContext already contains the SourceManager. llvm-svn: 155198 --- clang/unittests/Tooling/RecursiveASTVisitorTest.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/clang/unittests/Tooling/RecursiveASTVisitorTest.cpp b/clang/unittests/Tooling/RecursiveASTVisitorTest.cpp index d749f76a22bc..d7dad2738a05 100644 --- a/clang/unittests/Tooling/RecursiveASTVisitorTest.cpp +++ b/clang/unittests/Tooling/RecursiveASTVisitorTest.cpp @@ -38,7 +38,6 @@ public: protected: clang::ASTContext *Context; - clang::SourceManager *SM; private: class FindConsumer : public clang::ASTConsumer { @@ -59,7 +58,6 @@ private: virtual clang::ASTConsumer* CreateASTConsumer( clang::CompilerInstance& compiler, llvm::StringRef dummy) { - Visitor->SM = &compiler.getSourceManager(); Visitor->Context = &compiler.getASTContext(); /// TestConsumer will be deleted by the framework calling us. return new FindConsumer(Visitor); @@ -116,7 +114,7 @@ protected: // If we did not match, record information about partial matches. llvm::raw_string_ostream Stream(PartialMatches); Stream << ", partial match: \"" << Name << "\" at "; - Location.print(Stream, *this->SM); + Location.print(Stream, this->Context->getSourceManager()); } }