[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