[PATCH] D19664: [MemorySSA] Port to new pass manager

George Burgess IV via llvm-commits llvm-commits at lists.llvm.org
Tue May 31 17:31:02 PDT 2016


george.burgess.iv added inline comments.

================
Comment at: include/llvm/Transforms/Utils/MemorySSA.h:592
@@ -595,3 +591,3 @@
   // Memory SSA building info
-  MemorySSAWalker *Walker;
+  std::unique_ptr<MemorySSAWalker> Walker;
   unsigned NextID;
----------------
FWIW, I think Danny wasn't a massive fan of MSSA owning the caching walker. I don't fully recall why, though.


http://reviews.llvm.org/D19664





More information about the llvm-commits mailing list