[PATCH] D147866: ValueTracking: Implement computeKnownFPClass for minnum/maxnum

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D147866#4269348 <https://reviews.llvm.org/D147866#4269348>, @jcranmer-intel wrote:

> `minnum(0.0, 0.0)` can return `-0.0`, so the handling around positive/negative zero isn't correct.

How is it not correct? If either operand can be +/- 0 such is reported


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

https://reviews.llvm.org/D147866