[PATCH] Remove pimpl class from PassRegistry.
David Blaikie
dblaikie at gmail.com
Wed Jun 11 17:21:32 PDT 2014
Looks good to me, please commit! (various optional comments, curiosities, etc)
================
Comment at: include/llvm/PassInfo.h:1
@@ +1,2 @@
+//===- llvm/PassInfo.h - Pass Info class ------------------------*- C++ -*-===//
+//
----------------
I take it you had to split this out to resolve a dependency that wasn't exposed when everything was pmpld away?
================
Comment at: include/llvm/PassRegistry.h:61
@@ -39,3 +60,3 @@
public:
- PassRegistry() : pImpl(nullptr) { }
+ PassRegistry() { }
~PassRegistry();
----------------
You can remove this now that it's got nothing in it.
================
Comment at: lib/IR/PassRegistry.cpp:39
@@ -74,10 +38,3 @@
PassRegistry::~PassRegistry() {
}
----------------
delete this dtor now that it's not doing anything special?
================
Comment at: lib/IR/PassRegistry.cpp:71
@@ -117,3 +70,3 @@
- if (ShouldFree) Impl->ToFree.push_back(std::unique_ptr<const PassInfo>(&PI));
+ if (ShouldFree) ToFree.push_back(std::unique_ptr<const PassInfo>(&PI));
}
----------------
Oh dynamically conditional ownership... (not your problem, just me being sad)
http://reviews.llvm.org/D4107
More information about the llvm-commits
mailing list