[PATCH] D107806: [LoopFlatten] Fix assertion failure

Rosie Sumpter via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 10 02:21:02 PDT 2021


RosieSumpter created this revision.
RosieSumpter added reviewers: SjoerdMeijer, dmgreen, simon_tatham.
Herald added a subscriber: hiraditya.
RosieSumpter requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

There is an assertion failure in computeOverflowForUnsignedMul
(used in checkOverflow) due to the inner and outer trip counts
having different types. This occurs when the IV has been widened,
but the loop components are not successfully rediscovered.
This is fixed by some refactoring of the code in findLoopComponents
which identifies the trip count of the loop.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D107806

Files:
  llvm/lib/Transforms/Scalar/LoopFlatten.cpp
  llvm/test/Transforms/LoopFlatten/widen-iv.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D107806.365390.patch
Type: text/x-patch
Size: 6683 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210810/5b2d947d/attachment.bin>


More information about the llvm-commits mailing list