[PATCH] D104063: [SimplifyCFG] avoid crash on degenerate loop

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 15:06:03 PDT 2021


lebedev.ri added a comment.

In D104063#2811595 <https://reviews.llvm.org/D104063#2811595>, @lebedev.ri wrote:

> And what happens when there's more than one PHI node in the block?
> I think the extra missing check is `!isa<Instruction>(IfCond) || cast<Instruction>(IfCond)->getParent() != BB`.

And thinking about it more, this should even be `PHINode`, not `Instruction`.


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

https://reviews.llvm.org/D104063



More information about the llvm-commits mailing list