[PATCH] D62013: [InstSimplify] Add unary fneg to `fsub 0.0, (fneg X) ==> X` transform

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 10:14:22 PDT 2019


arsenm added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:4372
 
-  // fsub 0.0, (fsub 0.0, X) ==> X if signed zeros are ignored.
+  // fsub 0.0, (fneg X) ==> X if signed zeros are ignored.
   if (FMF.noSignedZeros() && match(Op0, m_AnyZeroFP()) &&
----------------
Should keep the original comment, and add another line for the fneg case since both are handled


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62013





More information about the llvm-commits mailing list