diff --git a/llvm/lib/Target/AArch64/AArch64InstrGISel.td b/llvm/lib/Target/AArch64/AArch64InstrGISel.td --- a/llvm/lib/Target/AArch64/AArch64InstrGISel.td +++ b/llvm/lib/Target/AArch64/AArch64InstrGISel.td @@ -146,6 +146,16 @@ let InOperandList = (ins type0:$src1, untyped_imm_0:$imm); } +// Represents an integer to FP conversion on the FPR bank. +def G_SITOF : AArch64GenericInstruction { + let OutOperandList = (outs type0:$dst); + let InOperandList = (ins type0:$src); +} +def G_UITOF : AArch64GenericInstruction { + let OutOperandList = (outs type0:$dst); + let InOperandList = (ins type0:$src); +} + def : GINodeEquiv; def : GINodeEquiv; def : GINodeEquiv; @@ -163,6 +173,8 @@ def : GINodeEquiv; def : GINodeEquiv; def : GINodeEquiv; +def : GINodeEquiv; +def : GINodeEquiv; def : GINodeEquiv; diff --git a/llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp b/llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp --- a/llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp +++ b/llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp @@ -1941,6 +1941,24 @@ I.getOperand(1).setReg(NewSrc.getReg(0)); return true; } + case TargetOpcode::G_UITOFP: + case TargetOpcode::G_SITOFP: { + // If both source and destination regbanks are FPR, then convert the opcode + // to G_SITOF so that the importer can select it to an fpr variant. + // Otherwise, it ends up matching an fpr/gpr variant and adding a cross-bank + // copy. + Register SrcReg = I.getOperand(1).getReg(); + if (MRI.getType(SrcReg).isVector()) + return false; + if (RBI.getRegBank(SrcReg, MRI, TRI)->getID() == AArch64::FPRRegBankID) { + if (I.getOpcode() == TargetOpcode::G_SITOFP) + I.setDesc(TII.get(AArch64::G_SITOF)); + else + I.setDesc(TII.get(AArch64::G_UITOF)); + return true; + } + return false; + } default: return false; } diff --git a/llvm/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir b/llvm/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir --- a/llvm/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir +++ b/llvm/test/CodeGen/AArch64/GlobalISel/select-fp-casts.mir @@ -218,7 +218,7 @@ ... --- -name: sitofp_s32_s32_fpr +name: sitofp_s32_s32_fpr_gpr legalized: true regBankSelected: true @@ -230,7 +230,7 @@ bb.0: liveins: $w0 - ; CHECK-LABEL: name: sitofp_s32_s32_fpr + ; CHECK-LABEL: name: sitofp_s32_s32_fpr_gpr ; CHECK: [[COPY:%[0-9]+]]:gpr32 = COPY $w0 ; CHECK: [[SCVTFUWSri:%[0-9]+]]:fpr32 = SCVTFUWSri [[COPY]] ; CHECK: $s0 = COPY [[SCVTFUWSri]] @@ -239,6 +239,50 @@ $s0 = COPY %1(s32) ... +--- +name: sitofp_s32_s32_fpr_fpr +legalized: true +regBankSelected: true + +registers: + - { id: 0, class: fpr } + - { id: 1, class: fpr } + +body: | + bb.0: + liveins: $s0 + + ; CHECK-LABEL: name: sitofp_s32_s32_fpr_fpr + ; CHECK: [[COPY:%[0-9]+]]:fpr32 = COPY $s0 + ; CHECK: [[SCVTFv1i32:%[0-9]+]]:fpr32 = SCVTFv1i32 [[COPY]] + ; CHECK: $s0 = COPY [[SCVTFv1i32]] + %0(s32) = COPY $s0 + %1(s32) = G_SITOFP %0 + $s0 = COPY %1(s32) +... + +--- +name: uitofp_s32_s32_fpr_fpr +legalized: true +regBankSelected: true + +registers: + - { id: 0, class: fpr } + - { id: 1, class: fpr } + +body: | + bb.0: + liveins: $s0 + + ; CHECK-LABEL: name: uitofp_s32_s32_fpr_fpr + ; CHECK: [[COPY:%[0-9]+]]:fpr32 = COPY $s0 + ; CHECK: [[UCVTFv1i32:%[0-9]+]]:fpr32 = UCVTFv1i32 [[COPY]] + ; CHECK: $s0 = COPY [[UCVTFv1i32]] + %0(s32) = COPY $s0 + %1(s32) = G_UITOFP %0 + $s0 = COPY %1(s32) +... + --- name: sitofp_s32_s64_fpr legalized: true