[PATCH] D89038: [PatternMatch] Add new FP matchers. NFC.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:3665
         continue;
-      auto *CElt = dyn_cast<ConstantFP>(Elt);
-      if (!CElt || CElt->isNaN())
+      if (!match(Elt, m_NonNaN()))
         return false;
----------------
Would it be better to simplify this entire loop to match(V, m_NonNaN()) ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89038