[PATCH] D32596: [DAGCombine] Transform (fadd A, (fmul B, -2.0)) -> (fsub A, (fadd B, B)).

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 12:48:47 PDT 2017


mcrosier added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:9468
+  // fold (fadd (fmul B, -2.0), A) -> (fsub A, (fadd B, B))
+  if (isFMulNegTwo(N0) || isFMulNegTwo(N1)) {
+    bool N1IsFMul = isFMulNegTwo(N1);
----------------
efriedma wrote:
> hasOneUse()?
Very good point!  One second.


https://reviews.llvm.org/D32596





More information about the llvm-commits mailing list