[PATCH] D59944: [DebugInfo] Fix pr41180 : Loop Vectorization Debugify Failure

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 1 12:07:16 PDT 2019


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

Thanks, lgtm.



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:2876
+  assert(isa<BranchInst>(OrigLoop->getLoopLatch()->getTerminator()) &&
+         "Scalar loop latch terminator isn't a branch");
+  BranchInst *ScalarLatchBr =
----------------
Consider adding a comment here explaining that the original loop must have a single latch block, because we already require it to have a single exit block?


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

https://reviews.llvm.org/D59944





More information about the llvm-commits mailing list