[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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 16 08:09:21 PDT 2021


spatel added a comment.

In D87479#2694818 <https://reviews.llvm.org/D87479#2694818>, @spatel wrote:

> https://godbolt.org/z/4vss8qrTG

The x86 asm also shows an interesting trade-off: SDAG could form a reciprocal estimate when the fdiv was sunk back into the loop...but it couldn't hoist the refinement math, so that's likely worse perf.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87479



More information about the llvm-commits mailing list