[PATCH] D94995: Loop peeling: check that latch is conditional branch

Joseph Tremoulet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 13:50:42 PST 2021


JosephTremoulet added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LoopPeel.cpp:664
   assert(PeelCount > 0 && "Attempt to peel out zero iterations?");
   assert(canPeel(L) && "Attempt to peel a loop which is not peelable?");
 
----------------
JosephTremoulet wrote:
> Sorry, I jumped the gun in posting this -- this change makes the assertion here fail in several of the loop peeling lit tests.  I'm taking a look...
> Sorry, I jumped the gun in posting this -- this change makes the assertion here fail in several of the loop peeling lit tests.  I'm taking a look...

Fixed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94995



More information about the llvm-commits mailing list