[PATCH] D93764: [LoopUnswitch] Implement first version of partial unswitching.

Fri Jan 29 14:11:27 PST 2021

fhahn added a comment.

In D93764#2527703 <https://reviews.llvm.org/D93764#2527703>, @uabelho wrote:

> Hi!
>
> I started seeing crashes with this patch:
>
> opt -o /dev/null bbi-52312.ll -loop-unswitch
>
> Result:
>
> opt: ../lib/Analysis/MemorySSAUpdater.cpp:1154: void llvm::MemorySSAUpdater::applyInsertUpdates(ArrayRef<llvm::CFGUpdate>, llvm::DominatorTree &, const GraphDiff<llvm::BasicBlock *> *): Assertion `IDom && "Block must have a valid IDom."' failed.
>
> F15216359: bbi-52312.ll <https://reviews.llvm.org/F15216359>

Thanks ,let me take a look!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93764