diff options
author | Craig Topper <craig.topper@gmail.com> | 2014-09-10 04:53:53 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@gmail.com> | 2014-09-10 04:53:53 +0000 |
commit | b8a705305562a8312e6d63ced9c8237d9fae21f3 (patch) | |
tree | 114deebfd15c4e4b9defd7aa4ae4b9b41bbe744e /clang/lib/Frontend/DependencyFile.cpp | |
parent | 7ff15929606a9004dba0c9a18371aa5438b927d2 (diff) | |
download | llvm-b8a705305562a8312e6d63ced9c8237d9fae21f3.zip llvm-b8a705305562a8312e6d63ced9c8237d9fae21f3.tar.gz llvm-b8a705305562a8312e6d63ced9c8237d9fae21f3.tar.bz2 |
Unique_ptrify PPCallbacks ownership.
Unique_ptr creation stil needs to be moved earlier at some of the call sites.
llvm-svn: 217474
Diffstat (limited to 'clang/lib/Frontend/DependencyFile.cpp')
-rw-r--r-- | clang/lib/Frontend/DependencyFile.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/clang/lib/Frontend/DependencyFile.cpp b/clang/lib/Frontend/DependencyFile.cpp index ce33086..22ed38f 100644 --- a/clang/lib/Frontend/DependencyFile.cpp +++ b/clang/lib/Frontend/DependencyFile.cpp @@ -121,7 +121,8 @@ bool DependencyCollector::sawDependency(StringRef Filename, bool FromModule, DependencyCollector::~DependencyCollector() { } void DependencyCollector::attachToPreprocessor(Preprocessor &PP) { - PP.addPPCallbacks(new DepCollectorPPCallbacks(*this, PP.getSourceManager())); + PP.addPPCallbacks( + llvm::make_unique<DepCollectorPPCallbacks>(*this, PP.getSourceManager())); } void DependencyCollector::attachToASTReader(ASTReader &R) { R.addListener(llvm::make_unique<DepCollectorASTListener>(*this)); @@ -203,7 +204,7 @@ DependencyFileGenerator *DependencyFileGenerator::CreateAndAttachToPreprocessor( PP.SetSuppressIncludeNotFoundError(true); DFGImpl *Callback = new DFGImpl(&PP, Opts); - PP.addPPCallbacks(Callback); // PP owns the Callback + PP.addPPCallbacks(std::unique_ptr<PPCallbacks>(Callback)); return new DependencyFileGenerator(Callback); } |