[PATCH] D29047: Introduce a basic MemorySSA updater, that supports insertDef and insertUse operations.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 15:05:17 PST 2017


davide accepted this revision.
davide added inline comments.
This revision is now accepted and ready to land.


================
Comment at: include/llvm/Transforms/Utils/MemorySSA.h:741-742
 
+class MemorySSAUpdater {
+private:
+  MemorySSA *MSSA;
----------------
dberlin wrote:
> davide wrote:
> > This is just my personal preference, so ignore it if you don't like, but, what about moving the updater to its own separate file?
> I'll do this as a followup (otherwise it's going to completely screw up the comments here :P)
Yeah, feel free to do that in a follow up =)


https://reviews.llvm.org/D29047





More information about the llvm-commits mailing list