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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to