[PATCH] D126774: [InstCombine] Use +0.0 instead of -0.0 as the FP identity for some folds

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 6 07:31:08 PDT 2022


david-arm added a comment.

In D126774#3553210 <https://reviews.llvm.org/D126774#3553210>, @dmgreen wrote:

> The Arm backend has patterns to turn the awkward `fadd(select(.., -0.0))` back into a more natural predicated fadd. They will need to be made to work with both the forms, if this is changing.

Ah! My mistake - was looking in the wrong backend - should be CodeGen/Thumb2. Doh!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126774



More information about the llvm-commits mailing list