[PATCH] D108476: [runtimeunroll] Support epilogue unrolling with a parent loop

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 2 10:42:38 PDT 2021


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

LGTM



================
Comment at: llvm/lib/Transforms/Utils/LoopUnrollRuntime.cpp:735
+    // loop. Note that this leaves loopinfo temporarily out of sync with the
+    // CFG until the actual epilogue loop is inserted.
+    if (auto *ParentL = L->getParentLoop())
----------------
Can we add an `LI.verify(DT)` at the end of this transform, to make sure `LI` is in sync with CFG?


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

https://reviews.llvm.org/D108476



More information about the llvm-commits mailing list