[PATCH] D143631: [LTO] Don't let InstCombine re-sink the vastly more expensive fdiv

Tue Feb 14 11:30:05 PST 2023

spatel added a comment.

In D143631#4126874 <https://reviews.llvm.org/D143631#4126874>, @craig.topper wrote:

> Do you have a proposed refinement for D87479 <https://reviews.llvm.org/D87479>? Turns out `LI` is only used by InstCombine if its available already in the pipeline so I don't know if it's safe to use that instead of the basic block check.

Yes, using LoopInfo was what I was thinking. I just made a rough draft of that, and it handles the motivating case here, and I think it would work on the other examples we've raised too. I can clean it up, test some more, and post it.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143631