[PATCH] D57221: [LoopSimplifyCFG] Change logic of dead loops removal to avoid hitting asserts

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 05:30:57 PST 2019


fedor.sergeev accepted this revision.
fedor.sergeev added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: lib/Transforms/Scalar/LoopSimplifyCFG.cpp:404
     }
+    for (auto *BB : DeadLoopBlocks)
+      if (LI.isLoopHeader(BB)) {
----------------
Please, put a short comment here describing the idea of this first loop through dead blocks.


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

https://reviews.llvm.org/D57221





More information about the llvm-commits mailing list