[PATCH] D72312: [DAGCombine] Fold the (fma -x, y, -z) to -(fma x, y, z)

qshanz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 7 03:32:24 PST 2020


steven.zhang added a comment.

This is really a good suggestion. I will go with that mechanism, though it seems that, it didn't respect the  TLI.isFNegFree() and UnsafeFPMath which didn't make sense. (When it is FNEG opcode, we just return 2 to indicate that it is profitable. What if some target is free for FNEG operations ...)


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

https://reviews.llvm.org/D72312





More information about the llvm-commits mailing list