[PATCH] D113584: [ARM] Fold (fadd x, (vselect c, y, -1.0)) into (vselect c, (fadd x, y), x)

Sam Tebbs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 15 02:11:59 PST 2021


samtebbs accepted this revision.
samtebbs added a comment.
This revision is now accepted and ready to land.

LGTM with one request



================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:16445
+
+  // Turn (fadd x, (vselect c, y, -1.0)) into (vselect c, (fadd x, y), x)
+  // The second form can be more easily turned into a predicated vadd, and
----------------
The -1.0 in the comment should be -0.0.


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

https://reviews.llvm.org/D113584



More information about the llvm-commits mailing list