[PATCH] Refactor reciprocal square root estimate into target-independent function (part 1).

Sanjay Patel spatel at rotateright.com
Sat Sep 20 15:24:36 PDT 2014


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:7012-7014
@@ -7007,1 +7011,5 @@
+    // it may be folded into a target-specific square root estimate intruction.
+    SDValue SqrtOp = BuildRSQRTE(N);
+    if (SqrtOp.getNode())
+      return SqrtOp;
   }
----------------
chandlerc wrote:
> You can make SqrtOp a condition variable now, I fixed SDValue to support this:
> 
>   if (SDValue SqrtOp = BuildRSQRTE(N))
>     return SqrtOp;
Nice! There's a lot more where that came from...

http://reviews.llvm.org/D5425






More information about the llvm-commits mailing list