Skip to content

Commit c22b64e

Browse files
author
Jessica Paquette
committed
[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.
1 parent b2e77cd commit c22b64e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -727,7 +727,8 @@ AArch64LegalizerInfo::AArch64LegalizerInfo(const AArch64Subtarget &ST)
727727
// TODO: Vector types.
728728
getActionDefinitionsBuilder({G_SADDSAT, G_SSUBSAT}).lowerIf(isScalar(0));
729729

730-
getActionDefinitionsBuilder(G_ISNAN).lower();
730+
getActionDefinitionsBuilder(G_ISNAN).lowerIf(
731+
any(isVector(1), typeInSet(1, {s16, s32, s64})));
731732

732733
// TODO: Vector types.
733734
getActionDefinitionsBuilder({G_FMAXNUM, G_FMINNUM})

0 commit comments

Comments
 (0)