[PATCH] D24808: [AArch64] Avoid materializing 0.0 when generating FP CSEL

Evandro Menezes via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 21 12:34:19 PDT 2016


evandro marked an inline comment as done.

================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:4083
@@ +4082,3 @@
+
+  if (DAG.getTarget().Options.UnsafeFPMath) {
+    // Transform "a == 0.0 ? 0.0 : x" to "a == 0.0 ? a : x" and
----------------
gberry wrote:
> Isn't the goal to check these options less and look at the DAG node flags instead?
> If so, this might be better written as follows?
>   DAG.getFlags()->hasNoSignedZeros()
Yes, thank you!


Repository:
  rL LLVM

https://reviews.llvm.org/D24808





More information about the llvm-commits mailing list