[PATCH] D147871: ValueTracking: Implement computeKnownFPClass for sqrt

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:4491
+            (InterestedClasses & fcNan) != fcNone &&
+            cannotBeOrderedLessThanZeroImpl(II->getArgOperand(0), TLI, false,
+                                            Depth + 1))
----------------
foad wrote:
> This should reuse info from the first call to computeKnownFPClass, instead of making a second call to cannotBeOrderedLessThanZeroImpl.
Oh right, this predates when I added cannotBeOrderedLessThanZero to KnownFPClass


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147871/new/

https://reviews.llvm.org/D147871