[PATCH] D72824: [X86] Add combination for fma and fneg on X86 under strict FP.
Simon Pilgrim via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 23 02:00:34 PST 2020
RKSimon added a comment.
In D72824#1835633 <https://reviews.llvm.org/D72824#1835633>, @pengfei wrote:
> In D72824#1835568 <https://reviews.llvm.org/D72824#1835568>, @RKSimon wrote:
>
> > Add to isNegatibleForFree/getNegatedExpression as well?
>
>
> We don't need to add them to isNegatibleForFree/getNegatedExpression. We won't negate any FMA results by negative FMA instructions under strict FP, because fneg(FMA) has different result from FNMA under some rounding mode.
Thanks for confirming - no more comments from me.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72824/new/
https://reviews.llvm.org/D72824
More information about the cfe-commits
mailing list