[PATCH] D127499: [InstCombine] Don't push operation across loop phi

[Sanjay Patel via Phabricator via llvm-commits]

spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - thanks!

Note: I think we should still do the narrowing transform that I suggested in D123408 <https://reviews.llvm.org/D123408>, but that's independent of this fix. That narrowing commit ran into its own infinite loop due to a conflict matching constant expressions, but I can fix that.


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

https://reviews.llvm.org/D127499