[PATCH] D68191: Simplify function llvm::removeUnreachableBlocks() to avoid (re-)computation.

Aditya Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 29 10:59:43 PDT 2019


hiraditya added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/Local.cpp:2255
+
+  for (auto *BB : DeadBlockSet) {
     if (DTU) {
----------------
With this patch, it now seems like we are iterating on DeadBlockSet twice, is it possible to merge the two loops?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68191





More information about the llvm-commits mailing list