From c22b64ef66f7518abb6f022fcdfd86d16c764caf Mon Sep 17 00:00:00 2001 From: Jessica Paquette Date: Wed, 18 Aug 2021 13:57:42 -0700 Subject: [PATCH] [AArch64][GlobalISel] Don't allow s128 for G_ISNAN getAPFloatFromSize doesn't support s128, so we can't lower this without asserting right now. To fix the buildbots, don't allow any scalars other than s16, s32, and s64. --- llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp b/llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp index bbc3df391da8..5320f44af2a3 100644 --- a/llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp +++ b/llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp @@ -727,7 +727,8 @@ AArch64LegalizerInfo::AArch64LegalizerInfo(const AArch64Subtarget &ST) // TODO: Vector types. getActionDefinitionsBuilder({G_SADDSAT, G_SSUBSAT}).lowerIf(isScalar(0)); - getActionDefinitionsBuilder(G_ISNAN).lower(); + getActionDefinitionsBuilder(G_ISNAN).lowerIf( + any(isVector(1), typeInSet(1, {s16, s32, s64}))); // TODO: Vector types. getActionDefinitionsBuilder({G_FMAXNUM, G_FMINNUM})