r215325 - Out-of-line CompilerInstance::takeSema to avoid a header dependence from CompilerInstance.h on Sema.h

David Blaikie dblaikie at gmail.com
Sun Aug 10 13:12:39 PDT 2014


Author: dblaikie
Date: Sun Aug 10 15:12:39 2014
New Revision: 215325

URL: http://llvm.org/viewvc/llvm-project?rev=215325&view=rev
Log:
Out-of-line CompilerInstance::takeSema to avoid a header dependence from CompilerInstance.h on Sema.h

Hopefully this fixes the libstdc++ build on some of the buildbots after
r215321.

Modified:
    cfe/trunk/include/clang/Frontend/CompilerInstance.h
    cfe/trunk/lib/Frontend/CompilerInstance.cpp

Modified: cfe/trunk/include/clang/Frontend/CompilerInstance.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/CompilerInstance.h?rev=215325&r1=215324&r2=215325&view=diff
==============================================================================
--- cfe/trunk/include/clang/Frontend/CompilerInstance.h (original)
+++ cfe/trunk/include/clang/Frontend/CompilerInstance.h Sun Aug 10 15:12:39 2014
@@ -460,7 +460,7 @@ public:
     return *TheSema;
   }
 
-  std::unique_ptr<Sema> takeSema() { return std::move(TheSema); }
+  std::unique_ptr<Sema> takeSema();
   void resetAndLeakSema() { BuryPointer(TheSema.release()); }
 
   /// }

Modified: cfe/trunk/lib/Frontend/CompilerInstance.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInstance.cpp?rev=215325&r1=215324&r2=215325&view=diff
==============================================================================
--- cfe/trunk/lib/Frontend/CompilerInstance.cpp (original)
+++ cfe/trunk/lib/Frontend/CompilerInstance.cpp Sun Aug 10 15:12:39 2014
@@ -108,7 +108,11 @@ void CompilerInstance::setASTConsumer(st
 void CompilerInstance::setCodeCompletionConsumer(CodeCompleteConsumer *Value) {
   CompletionConsumer.reset(Value);
 }
- 
+
+std::unique_ptr<Sema> CompilerInstance::takeSema() {
+  return std::move(TheSema);
+}
+
 IntrusiveRefCntPtr<ASTReader> CompilerInstance::getModuleManager() const {
   return ModuleManager;
 }





More information about the cfe-commits mailing list