[InstSimplify] enhance fcmp fold with never-nan operand
authorSanjay Patel <spatel@rotateright.com>
Sat, 8 Jun 2019 15:12:33 +0000 (15:12 +0000)
committerSanjay Patel <spatel@rotateright.com>
Sat, 8 Jun 2019 15:12:33 +0000 (15:12 +0000)
commit4329c15f11759b94b7739a7860b8fd141598c052
tree767b7317b8b495edd6ee76a812505a94dc27fa82
parent375297f38fe87034e4f35c2b1115922d3773ced5
[InstSimplify] enhance fcmp fold with never-nan operand

This is 1 step towards correcting our usage of fast-math-flags when applied on an fcmp.
In this case, we are checking for 'nnan' on the fcmp itself rather than the operand of
the fcmp. But I'm leaving that clause in until we're more confident that we can stop
relying on fcmp's FMF.

By using the more general "isKnownNeverNaN()", we gain a simplification shown on the
tests with 'uitofp' regardless of the FMF on the fcmp (uitofp never produces a NaN).
On the tests with 'fabs', we are now relying on the FMF for the call fabs instruction
in addition to the FMF on the fcmp.

I'll update the 'ult' case below here as a follow-up assuming no problems here.

Differential Revision: https://reviews.llvm.org/D62979

llvm-svn: 362879
llvm/lib/Analysis/InstructionSimplify.cpp
llvm/test/Transforms/InstSimplify/floating-point-compare.ll