[PATCH] D67637: [MemorySSA] Fix phi insertion when inserting a def.

George Burgess IV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 20:52:04 PDT 2019


george.burgess.iv accepted this revision.
george.burgess.iv added a comment.
This revision is now accepted and ready to land.

thanks!



================
Comment at: lib/Analysis/MemorySSAUpdater.cpp:343
+      for (auto &VH : InsertedPHIs)
+        if (auto *RealPHI = cast_or_null<MemoryPhi>(VH))
+          DefiningBlocks.insert(RealPHI->getBlock());
----------------
will the compiler accept a `const` here? (and above)


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67637/new/

https://reviews.llvm.org/D67637





More information about the llvm-commits mailing list