[llvm] [InstCombine] Implement `fcmp (fadd x, 0.0), y` => `fcmp x, y` optimization (PR #88476)
Yingwei Zheng via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 12 00:15:35 PDT 2024
dtcxzyw wrote:
> 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?
Yeah, `fsub x, +/-0.0` are not in canonical form. I don't think these tests are necessary.
https://github.com/llvm/llvm-project/pull/88476
More information about the llvm-commits
mailing list