[PATCH] D143195: ValueTracking: Add start of computeKnownFPClass API

[Serge Pavlov via Phabricator via llvm-commits]

sepavloff added inline comments.


================
Comment at: llvm/include/llvm/ADT/FloatingPointMode.h:255
 
+inline FPClassTest fneg(FPClassTest Mask) {
+  FPClassTest NewMask = Mask & fcNan;
----------------
nikic wrote:
> Commit these NFC changes separately? It would be nice if they weren't part of the header though.
Both these functions require documentation about its meaning.


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

https://reviews.llvm.org/D143195