[PATCH] D47149: [SCCP] Mark DominatorTreeAnalysis as preserved.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Right, IPSCCP does use changeToUnreachable, but it can be told to update the DT. The thing that confused me is that we use removeAllNonTerminatorInstruction... in runSCCP with a comment that we should not change the CFG. Is there a reason we use changeToUnreachable in IPSCCP but removeAllNonTerminatorInstruction... in SCCP?


https://reviews.llvm.org/D47149