[PATCH] D29046: MemorySSA: Link all defs together into an intrusive defslist, to make updater easier

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 11:53:51 PST 2017


dberlin marked an inline comment as done.
dberlin added inline comments.


================
Comment at: lib/Transforms/Utils/MemorySSA.cpp:1490
     AccessList *Accesses = getOrCreateAccessList(BB);
+    DefsList *Defs = getOrCreateDefsList(BB);
     MemoryPhi *Phi = new MemoryPhi(BB->getContext(), BB, NextID++);
----------------
george.burgess.iv wrote:
> `insertIntoListsForBlock`?
I just used createMemoryPhi instead


https://reviews.llvm.org/D29046





More information about the llvm-commits mailing list