[PATCH] D56284: [UnrollRuntime] Fix domTree failure in multiexit unrolling

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 4 10:18:04 PST 2019


anna updated this revision to Diff 180269.
anna added a comment.

The IDom can be any block within the loop (as mentioned in the comment).
So, we need to check for IDom *contained* in the loop.
Added test case shows where the IDom is within the inner loop of the original loop.
Both test cases pass now with the fix.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56284

Files:
  lib/Transforms/Utils/LoopUnrollRuntime.cpp
  test/Transforms/LoopUnroll/runtime-loop-multiexit-dom-verify.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56284.180269.patch
Type: text/x-patch
Size: 5212 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190104/4c050a07/attachment.bin>


More information about the llvm-commits mailing list