[PATCH] D87479: [InstCombine] Don't sink the fdiv from (fmul (fdiv 1.0, %x), %y) if the fdiv isn't in the same basic block as the fmul

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

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

> When does that happen?
> `-O3` seems to be fine: https://godbolt.org/z/1K3r56
> Do we have InstCombine invocations after last LICM?

We found it in our internal code base so maybe there’s something different about our pipeline. But still seems like we shouldn’t have two passes making opposite choices.


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

https://reviews.llvm.org/D87479