[PATCH] D57428: [SCEV] Guard movement of insertion point for loop-invariants (take 2)

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy added a comment.

Apologies for the late review Warren.

I agree that it is weird that the insert point is a PHI node.  Have you traced through the code to see how we're ending up in that situation?  Is IndVarSimplify choosing this insertion point?


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

https://reviews.llvm.org/D57428