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

George Burgess IV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 12:54:12 PST 2017


george.burgess.iv 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++);
----------------
dberlin wrote:
> george.burgess.iv wrote:
> > `insertIntoListsForBlock`?
> I just used createMemoryPhi instead
WFM


https://reviews.llvm.org/D29046





More information about the llvm-commits mailing list