tree 2a9bbb9be084a40959772db15a99096a693420d2
parent a3b1fcbc0cf5b70015d0f8aa983263d1ca84a8c8
author David Stenberg <david.stenberg@ericsson.com> 1606382159 +0100
committer David Stenberg <david.stenberg@ericsson.com> 1606397301 +0100

[IndVarSimplify] Fix Modified status when handling dead PHI nodes

When bailing out in rewriteLoopExitValues() you could be left with PHI
nodes in the DeadInsts vector. Those would be not handled by the use of
RecursivelyDeleteTriviallyDeadInstructions() in IndVarSimplify. This
resulted in the IndVarSimplify pass returning an incorrect modified
status. This was caught by the expensive check introduced in D86589.

This patches changes IndVarSimplify so that it deletes those PHI nodes,
using RecursivelyDeleteDeadPHINode().

This fixes PR47486.

Reviewed By: mkazantsev

Differential Revision: https://reviews.llvm.org/D91153