[PATCH] D89038: [PatternMatch] Add new FP matchers. NFC.
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 22 08:47:22 PDT 2020
lebedev.ri added a comment.
Anyways i think this looks fine to me now. Thanks.
================
Comment at: llvm/include/llvm/IR/PatternMatch.h:616
+/// Match a finite FP constant, i.e. not infinity or NaN.
+/// For vectors, this includes constants with undefined elements.
+inline cstfp_pred_ty<is_finite> m_Finite() {
----------------
foad wrote:
> lebedev.ri wrote:
> > So if this is using `apf_pred_ty`, then this doesn't support `undef` constants then?
> Correct. m_Finite() does, m_Finite(const APFloat *&) does not. This is the pre-existing problem I mentioned which has now been copied across from the integer matchers to the fp ones.
But it says it does.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89038/new/
https://reviews.llvm.org/D89038
More information about the llvm-commits
mailing list