[PATCH] Remove pimpl class from PassRegistry.

Zachary Turner zturner at google.com
Wed Jun 11 17:28:01 PDT 2014


================
Comment at: include/llvm/PassInfo.h:1
@@ +1,2 @@
+//===- llvm/PassInfo.h - Pass Info class ------------------------*- C++ -*-===//
+//
----------------
David Blaikie wrote:
> I take it you had to split this out to resolve a dependency that wasn't exposed when everything was pmpld away?
Yes, I actually updated the description on Phab, but it didn't send out an email notification for that.  Basically, the pimpl has a unique_ptr<PassInfo>, and PassSupport.h already includes PassRegistry.h, so I couldn't have PassRegistry.h include PassSupport.h to get this class.

Plus, one class-one file is just better when the classes are non-trivial :)

http://reviews.llvm.org/D4107






More information about the llvm-commits mailing list