[PATCH] D48790: Update MemorySSA in Local utils removing blocks.

Chijun Sima via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 15 08:21:07 PDT 2018


NutshellySima added a comment.

Looks like a nice improvement.



================
Comment at: lib/Transforms/Utils/Local.cpp:2231
 
+  // FIXME: May reuse this set below if deletion order does not matter.
+  SmallPtrSet<BasicBlock *, 16> DeadBlockSet;
----------------
Yes, I think the deletion order doesn't matter.


Repository:
  rL LLVM

https://reviews.llvm.org/D48790





More information about the llvm-commits mailing list