[PATCH] D43765: [InstSimplify] loosen FMF for sqrt(X) * sqrt(X) --> X

Steve Canon via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 5 08:56:03 PST 2018


scanon added a comment.

IIRC `Intrinsic::sqrt` is undef for negative inputs (unlike the `sqrt` libcall), so we don't need `FMF.noNaNs` to license this transformation.


https://reviews.llvm.org/D43765





More information about the llvm-commits mailing list