[llvm] [InstCombine] Implement `fcmp (fadd x, 0.0), y` => `fcmp x, y` optimization (PR #88476)

Vlad Mishel via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 11 23:38:25 PDT 2024


vmishelcs wrote:

@dtcxzyw Please take a look whenever you have time :) Let me know if I should make any changes.

Question: Should I also add tests for the case of `fcmp (fsub x, 0.0), y`? From what I can tell my implementation already covers this case. Is this because somewhere in the code we transform `fsub x, 0.0` => `fadd x, 0.0`?

https://github.com/llvm/llvm-project/pull/88476


More information about the llvm-commits mailing list