[llvm] r262316 - [mips] Promote the result of SETCC nodes to GPR width.
Vasileios Kalintiris via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 1 12:33:50 PST 2016
Hi Evgenii,
I reverted this in r262387. If you could provide me with a test-case it would be quite useful.
Thanks,
Vasileios
________________________________________
From: Evgenii Stepanov [eugeni.stepanov at gmail.com]
Sent: 01 March 2016 20:02
To: Vasileios Kalintiris; Nico Weber
Cc: LLVM Commits
Subject: Re: [llvm] r262316 - [mips] Promote the result of SETCC nodes to GPR width.
Hi,
you broke x86_64 with this commit. I'm reducing the testcase now.
See:
https://build.chromium.org/p/chromium.fyi/builders/ClangToTLinux/builds/4488/steps/compile/logs/stdio
On Tue, Mar 1, 2016 at 2:08 AM, Vasileios Kalintiris via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: vkalintiris
> Date: Tue Mar 1 04:08:01 2016
> New Revision: 262316
>
> URL: http://llvm.org/viewvc/llvm-project?rev=262316&view=rev
> Log:
> [mips] Promote the result of SETCC nodes to GPR width.
>
> Summary:
> This patch modifies the existing comparison, branch, conditional-move
> and select patterns, and adds new ones where needed. Also, the updated
> SLT{u,i,iu} set of instructions generate a GPR width result.
>
> The majority of the code changes in the Mips back-end fix the wrong
> assumption that the result of SETCC nodes always produce an i32 value.
> The changes in the common code path account for the fact that in 64-bit
> MIPS targets, i1 is promoted to i32 instead of i64.
>
> Reviewers: dsanders
>
> Subscribers: dsanders, llvm-commits
>
> Differential Revision: http://reviews.llvm.org/D10970
>
> Modified:
> llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
> llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
> llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
> llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
> llvm/trunk/lib/Target/Mips/Disassembler/MipsDisassembler.cpp
> llvm/trunk/lib/Target/Mips/MicroMips32r6InstrInfo.td
> llvm/trunk/lib/Target/Mips/MicroMips64r6InstrInfo.td
> llvm/trunk/lib/Target/Mips/MicroMipsInstrInfo.td
> llvm/trunk/lib/Target/Mips/Mips32r6InstrInfo.td
> llvm/trunk/lib/Target/Mips/Mips64InstrInfo.td
> llvm/trunk/lib/Target/Mips/Mips64r6InstrInfo.td
> llvm/trunk/lib/Target/Mips/MipsCondMov.td
> llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
> llvm/trunk/lib/Target/Mips/MipsISelLowering.h
> llvm/trunk/lib/Target/Mips/MipsInstrInfo.td
> llvm/trunk/lib/Target/Mips/MipsRegisterInfo.td
> llvm/trunk/lib/Target/Mips/MipsSEISelDAGToDAG.cpp
> llvm/trunk/lib/Target/Mips/MipsSEISelLowering.cpp
> llvm/trunk/test/CodeGen/Mips/atomic.ll
> llvm/trunk/test/CodeGen/Mips/blez_bgez.ll
> llvm/trunk/test/CodeGen/Mips/cmov.ll
> llvm/trunk/test/CodeGen/Mips/countleading.ll
> llvm/trunk/test/CodeGen/Mips/fcmp.ll
> llvm/trunk/test/CodeGen/Mips/llvm-ir/select-dbl.ll
> llvm/trunk/test/CodeGen/Mips/llvm-ir/select-flt.ll
> llvm/trunk/test/CodeGen/Mips/llvm-ir/select-int.ll
> llvm/trunk/test/CodeGen/Mips/octeon.ll
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Tue Mar 1 04:08:01 2016
> @@ -14151,8 +14151,7 @@ SDValue DAGCombiner::SimplifySelectCC(SD
> Temp = DAG.getZeroExtendInReg(SCC, SDLoc(N2),
> N2.getValueType());
> else
> - Temp = DAG.getNode(ISD::ZERO_EXTEND, SDLoc(N2),
> - N2.getValueType(), SCC);
> + Temp = DAG.getZExtOrTrunc(SCC, SDLoc(N2), N2.getValueType());
> } else {
> SCC = DAG.getSetCC(SDLoc(N0), MVT::i1, N0, N1, CC);
> Temp = DAG.getNode(ISD::ZERO_EXTEND, SDLoc(N2),
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp Tue Mar 1 04:08:01 2016
> @@ -520,7 +520,7 @@ SDValue DAGTypeLegalizer::PromoteIntRes_
> /// Promote the overflow flag of an overflowing arithmetic node.
> SDValue DAGTypeLegalizer::PromoteIntRes_Overflow(SDNode *N) {
> // Simply change the return type of the boolean result.
> - EVT NVT = TLI.getTypeToTransformTo(*DAG.getContext(), N->getValueType(1));
> + EVT NVT = getSetCCResultType(N->getValueType(1));
> EVT ValueVTs[] = { N->getValueType(0), NVT };
> SDValue Ops[] = { N->getOperand(0), N->getOperand(1) };
> SDValue Res = DAG.getNode(N->getOpcode(), SDLoc(N),
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp Tue Mar 1 04:08:01 2016
> @@ -750,8 +750,9 @@ void DAGTypeLegalizer::ReplaceValueWith(
> }
>
> void DAGTypeLegalizer::SetPromotedInteger(SDValue Op, SDValue Result) {
> - assert(Result.getValueType() ==
> - TLI.getTypeToTransformTo(*DAG.getContext(), Op.getValueType()) &&
> + assert(((Result.getValueType() ==
> + TLI.getTypeToTransformTo(*DAG.getContext(), Op.getValueType())) ||
> + (Result.getValueType() == getSetCCResultType(Op.getValueType()))) &&
> "Invalid type for promoted integer");
> AnalyzeNewValue(Result);
>
>
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp Tue Mar 1 04:08:01 2016
> @@ -3656,6 +3656,9 @@ void SelectionDAGBuilder::visitMaskedGat
> }
>
> void SelectionDAGBuilder::visitAtomicCmpXchg(const AtomicCmpXchgInst &I) {
> + const TargetLowering &TLI = DAG.getTargetLoweringInfo();
> + LLVMContext &Ctx = *DAG.getContext();
> +
> SDLoc dl = getCurSDLoc();
> AtomicOrdering SuccessOrder = I.getSuccessOrdering();
> AtomicOrdering FailureOrder = I.getFailureOrdering();
> @@ -3664,7 +3667,14 @@ void SelectionDAGBuilder::visitAtomicCmp
> SDValue InChain = getRoot();
>
> MVT MemVT = getValue(I.getCompareOperand()).getSimpleValueType();
> - SDVTList VTs = DAG.getVTList(MemVT, MVT::i1, MVT::Other);
> + EVT CCVT = TLI.getSetCCResultType(DAG.getDataLayout(), Ctx, MVT::i1);
> +
> + // Only use the result of getSetCCResultType if it is legal,
> + // otherwise just use the promoted result type (NVT).
> + if (!TLI.isTypeLegal(CCVT))
> + CCVT = TLI.getTypeToTransformTo(Ctx, MVT::i1);
> +
> + SDVTList VTs = DAG.getVTList(MemVT, CCVT, MVT::Other);
> SDValue L = DAG.getAtomicCmpSwap(
> ISD::ATOMIC_CMP_SWAP_WITH_SUCCESS, dl, MemVT, VTs, InChain,
> getValue(I.getPointerOperand()), getValue(I.getCompareOperand()),
>
> Modified: llvm/trunk/lib/Target/Mips/Disassembler/MipsDisassembler.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/Disassembler/MipsDisassembler.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/Disassembler/MipsDisassembler.cpp (original)
> +++ llvm/trunk/lib/Target/Mips/Disassembler/MipsDisassembler.cpp Tue Mar 1 04:08:01 2016
> @@ -124,9 +124,13 @@ static DecodeStatus DecodeFCCRegisterCla
> uint64_t Address,
> const void *Decoder);
>
> -static DecodeStatus DecodeFGRCCRegisterClass(MCInst &Inst, unsigned RegNo,
> - uint64_t Address,
> - const void *Decoder);
> +static DecodeStatus DecodeFGRCC32RegisterClass(MCInst &Inst, unsigned RegNo,
> + uint64_t Address,
> + const void *Decoder);
> +
> +static DecodeStatus DecodeFGRCC64RegisterClass(MCInst &Inst, unsigned RegNo,
> + uint64_t Address,
> + const void *Decoder);
>
> static DecodeStatus DecodeHWRegsRegisterClass(MCInst &Inst,
> unsigned Insn,
> @@ -898,6 +902,17 @@ DecodeStatus MipsDisassembler::getInstru
> if (Result == MCDisassembler::Fail)
> return MCDisassembler::Fail;
>
> + if (hasMips32r6() && isGP64()) {
> + DEBUG(dbgs() << "Trying MicroMipsR6_GP64 table (32-bit instructions):\n");
> + // Calling the auto-generated decoder function.
> + Result = decodeInstruction(DecoderTableMicroMipsR6_GP6432, Instr, Insn,
> + Address, this, STI);
> + if (Result != MCDisassembler::Fail) {
> + Size = 4;
> + return Result;
> + }
> + }
> +
> if (hasMips32r6()) {
> DEBUG(dbgs() << "Trying MicroMips32r632 table (32-bit instructions):\n");
> // Calling the auto-generated decoder function.
> @@ -940,9 +955,9 @@ DecodeStatus MipsDisassembler::getInstru
> }
>
> if (hasMips32r6() && isGP64()) {
> - DEBUG(dbgs() << "Trying Mips32r6_64r6 (GPR64) table (32-bit opcodes):\n");
> - Result = decodeInstruction(DecoderTableMips32r6_64r6_GP6432, Instr, Insn,
> - Address, this, STI);
> + DEBUG(dbgs() << "Trying Mips64r6 (GPR64) table (32-bit opcodes):\n");
> + Result = decodeInstruction(DecoderTableMips64r632, Instr, Insn, Address,
> + this, STI);
> if (Result != MCDisassembler::Fail) {
> Size = 4;
> return Result;
> @@ -1121,13 +1136,24 @@ static DecodeStatus DecodeFCCRegisterCla
> return MCDisassembler::Success;
> }
>
> -static DecodeStatus DecodeFGRCCRegisterClass(MCInst &Inst, unsigned RegNo,
> - uint64_t Address,
> - const void *Decoder) {
> +static DecodeStatus DecodeFGRCC32RegisterClass(MCInst &Inst, unsigned RegNo,
> + uint64_t Address,
> + const void *Decoder) {
> + if (RegNo > 31)
> + return MCDisassembler::Fail;
> +
> + unsigned Reg = getReg(Decoder, Mips::FGRCC32RegClassID, RegNo);
> + Inst.addOperand(MCOperand::createReg(Reg));
> + return MCDisassembler::Success;
> +}
> +
> +static DecodeStatus DecodeFGRCC64RegisterClass(MCInst &Inst, unsigned RegNo,
> + uint64_t Address,
> + const void *Decoder) {
> if (RegNo > 31)
> return MCDisassembler::Fail;
>
> - unsigned Reg = getReg(Decoder, Mips::FGRCCRegClassID, RegNo);
> + unsigned Reg = getReg(Decoder, Mips::FGRCC64RegClassID, RegNo);
> Inst.addOperand(MCOperand::createReg(Reg));
> return MCDisassembler::Success;
> }
>
> Modified: llvm/trunk/lib/Target/Mips/MicroMips32r6InstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MicroMips32r6InstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MicroMips32r6InstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/MicroMips32r6InstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -634,71 +634,71 @@ class CVT_S_L_MMR6_DESC : CVT_MMR6_DESC_
> II_CVT>, FGR_64;
>
> multiclass CMP_CC_MMR6<bits<6> format, string Typestr,
> - RegisterOperand FGROpnd> {
> + RegisterOperand FGRCCOpnd, RegisterOperand FGROpnd> {
> def CMP_AF_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.af.", Typestr), format, FIELD_CMP_COND_AF>,
> - CMP_CONDN_DESC_BASE<"af", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"af", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_UN_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.un.", Typestr), format, FIELD_CMP_COND_UN>,
> - CMP_CONDN_DESC_BASE<"un", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"un", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_EQ_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.eq.", Typestr), format, FIELD_CMP_COND_EQ>,
> - CMP_CONDN_DESC_BASE<"eq", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"eq", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_UEQ_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.ueq.", Typestr), format, FIELD_CMP_COND_UEQ>,
> - CMP_CONDN_DESC_BASE<"ueq", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"ueq", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_LT_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.lt.", Typestr), format, FIELD_CMP_COND_LT>,
> - CMP_CONDN_DESC_BASE<"lt", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"lt", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_ULT_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.ult.", Typestr), format, FIELD_CMP_COND_ULT>,
> - CMP_CONDN_DESC_BASE<"ult", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"ult", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_LE_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.le.", Typestr), format, FIELD_CMP_COND_LE>,
> - CMP_CONDN_DESC_BASE<"le", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"le", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_ULE_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.ule.", Typestr), format, FIELD_CMP_COND_ULE>,
> - CMP_CONDN_DESC_BASE<"ule", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"ule", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SAF_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.saf.", Typestr), format, FIELD_CMP_COND_SAF>,
> - CMP_CONDN_DESC_BASE<"saf", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"saf", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SUN_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.sun.", Typestr), format, FIELD_CMP_COND_SUN>,
> - CMP_CONDN_DESC_BASE<"sun", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"sun", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SEQ_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.seq.", Typestr), format, FIELD_CMP_COND_SEQ>,
> - CMP_CONDN_DESC_BASE<"seq", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"seq", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SUEQ_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.sueq.", Typestr), format, FIELD_CMP_COND_SUEQ>,
> - CMP_CONDN_DESC_BASE<"sueq", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"sueq", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SLT_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.slt.", Typestr), format, FIELD_CMP_COND_SLT>,
> - CMP_CONDN_DESC_BASE<"slt", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"slt", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SULT_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.sult.", Typestr), format, FIELD_CMP_COND_SULT>,
> - CMP_CONDN_DESC_BASE<"sult", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"sult", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SLE_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.sle.", Typestr), format, FIELD_CMP_COND_SLE>,
> - CMP_CONDN_DESC_BASE<"sle", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"sle", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> def CMP_SULE_#NAME : POOL32F_CMP_FM<
> !strconcat("cmp.sule.", Typestr), format, FIELD_CMP_COND_SULE>,
> - CMP_CONDN_DESC_BASE<"sule", Typestr, FGROpnd>, HARDFLOAT, R6MMR6Rel,
> - ISA_MICROMIPS32R6;
> + CMP_CONDN_DESC_BASE<"sule", Typestr, FGRCCOpnd, FGROpnd>,
> + HARDFLOAT, R6MMR6Rel, ISA_MICROMIPS32R6;
> }
>
> class ABSS_FT_MMR6_DESC_BASE<string instr_asm, RegisterOperand DstRC,
> @@ -763,8 +763,8 @@ class ROUND_W_S_MMR6_DESC : ABSS_FT_MMR6
> class ROUND_W_D_MMR6_DESC : ABSS_FT_MMR6_DESC_BASE<"round.w.d", FGR64Opnd,
> FGR64Opnd, II_ROUND>;
>
> -class SEL_S_MMR6_DESC : COP1_SEL_DESC_BASE<"sel.s", FGR32Opnd>;
> -class SEL_D_MMR6_DESC : COP1_SEL_DESC_BASE<"sel.d", FGR64Opnd> {
> +class SEL_S_MMR6_DESC : COP1_SEL_DESC_BASE<"sel.s", FGRCC32Opnd, FGR32Opnd>;
> +class SEL_D_MMR6_DESC : COP1_SEL_DESC_BASE<"sel.d", FGRCC32Opnd, FGR64Opnd> {
> // We must insert a SUBREG_TO_REG around $fd_in
> bit usesCustomInserter = 1;
> }
> @@ -1115,8 +1115,8 @@ def CVT_S_W_MMR6 : StdMMR6Rel, CVT_S_W_M
> ISA_MICROMIPS32R6;
> def CVT_S_L_MMR6 : StdMMR6Rel, CVT_S_L_MMR6_ENC, CVT_S_L_MMR6_DESC,
> ISA_MICROMIPS32R6;
> -defm S_MMR6 : CMP_CC_MMR6<0b000101, "s", FGR32Opnd>;
> -defm D_MMR6 : CMP_CC_MMR6<0b010101, "d", FGR64Opnd>;
> +defm S_MMR6 : CMP_CC_MMR6<0b000101, "s", FGRCC32Opnd, FGR32Opnd>;
> +defm D_MMR6 : CMP_CC_MMR6<0b010101, "d", FGRCC32Opnd, FGR64Opnd>;
> def ABS_S_MMR6 : StdMMR6Rel, ABS_S_MMR6_ENC, ABS_S_MMR6_DESC, ISA_MICROMIPS32R6;
> def ABS_D_MMR6 : StdMMR6Rel, ABS_D_MMR6_ENC, ABS_D_MMR6_DESC, ISA_MICROMIPS32R6;
> def FLOOR_L_S_MMR6 : StdMMR6Rel, FLOOR_L_S_MMR6_ENC, FLOOR_L_S_MMR6_DESC,
> @@ -1200,8 +1200,10 @@ def ROUND_W_S_MMR6 : StdMMR6Rel, ROUND_W
> ISA_MICROMIPS32R6;
> def ROUND_W_D_MMR6 : StdMMR6Rel, ROUND_W_D_MMR6_ENC, ROUND_W_D_MMR6_DESC,
> ISA_MICROMIPS32R6;
> -def SEL_S_MMR6 : StdMMR6Rel, SEL_S_MMR6_ENC, SEL_S_MMR6_DESC, ISA_MICROMIPS32R6;
> -def SEL_D_MMR6 : StdMMR6Rel, SEL_D_MMR6_ENC, SEL_D_MMR6_DESC, ISA_MICROMIPS32R6;
> +def SEL_S_MMR6 : StdMMR6Rel, SEL_S_MMR6_ENC, SEL_S_MMR6_DESC, ISA_MICROMIPS32R6,
> + HARDFLOAT;
> +def SEL_D_MMR6 : StdMMR6Rel, SEL_D_MMR6_ENC, SEL_D_MMR6_DESC, ISA_MICROMIPS32R6,
> + HARDFLOAT;
> def SELEQZ_S_MMR6 : StdMMR6Rel, SELEQZ_S_MMR6_ENC, SELEQZ_S_MMR6_DESC,
> ISA_MICROMIPS32R6;
> def SELEQZ_D_MMR6 : StdMMR6Rel, SELEQZ_D_MMR6_ENC, SELEQZ_D_MMR6_DESC,
>
> Modified: llvm/trunk/lib/Target/Mips/MicroMips64r6InstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MicroMips64r6InstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MicroMips64r6InstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/MicroMips64r6InstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -99,6 +99,12 @@ class DINSM_MM64R6_DESC : InsBase<"dinsm
> class DINS_MM64R6_DESC : InsBase<"dins", GPR64Opnd, uimm5, uimm5_inssize_plus1,
> MipsIns>;
>
> +class SEL_S_MM64R6_DESC : COP1_SEL_DESC_BASE<"sel.s", FGRCC64Opnd, FGR32Opnd> {
> + // We must copy FGRCC64Opnd's subreg into FGR32.
> + bit usesCustomInserter = 1;
> +}
> +class SEL_D_MM64R6_DESC : COP1_SEL_DESC_BASE<"sel.d", FGRCC64Opnd, FGR64Opnd>;
> +
> //===----------------------------------------------------------------------===//
> //
> // Instruction Definitions
> @@ -132,3 +138,10 @@ let DecoderNamespace = "MicroMipsR6" in
> def DINS_MM64R6: R6MMR6Rel, DINS_MM64R6_DESC, DINS_MM64R6_ENC,
> ISA_MICROMIPS64R6;
> }
> +
> +let DecoderNamespace = "MicroMipsR6_GP64" in {
> +def SEL_S_MM64R6 : StdMMR6Rel, SEL_S_MMR6_ENC, SEL_S_MM64R6_DESC,
> + ISA_MICROMIPS64R6, HARDFLOAT;
> +def SEL_D_MM64R6 : StdMMR6Rel, SEL_D_MMR6_ENC, SEL_D_MM64R6_DESC,
> + ISA_MICROMIPS64R6, HARDFLOAT;
> +}
>
> Modified: llvm/trunk/lib/Target/Mips/MicroMipsInstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MicroMipsInstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MicroMipsInstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/MicroMipsInstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -671,10 +671,10 @@ let DecoderNamespace = "MicroMips", Pred
> ADDI_FM_MM<0xc>;
> def ADDi_MM : MMRel, ArithLogicI<"addi", simm16, GPR32Opnd>,
> ADDI_FM_MM<0x4>;
> - def SLTi_MM : MMRel, SetCC_I<"slti", setlt, simm16, immSExt16, GPR32Opnd>,
> - SLTI_FM_MM<0x24>;
> - def SLTiu_MM : MMRel, SetCC_I<"sltiu", setult, simm16, immSExt16, GPR32Opnd>,
> - SLTI_FM_MM<0x2c>;
> + def SLTi_MM : MMRel, SetCC_I<"slti", setlt, simm16, immSExt16, GPR32Opnd,
> + GPR32Opnd>, SLTI_FM_MM<0x24>;
> + def SLTiu_MM : MMRel, SetCC_I<"sltiu", setult, simm16, immSExt16, GPR32Opnd,
> + GPR32Opnd>, SLTI_FM_MM<0x2c>;
> def ANDi_MM : MMRel, ArithLogicI<"andi", uimm16, GPR32Opnd>,
> ADDI_FM_MM<0x34>;
> def ORi_MM : MMRel, ArithLogicI<"ori", uimm16, GPR32Opnd>,
> @@ -694,8 +694,9 @@ let DecoderNamespace = "MicroMips", Pred
> def MUL_MM : MMRel, ArithLogicR<"mul", GPR32Opnd>, ADD_FM_MM<0, 0x210>;
> def ADD_MM : MMRel, ArithLogicR<"add", GPR32Opnd>, ADD_FM_MM<0, 0x110>;
> def SUB_MM : MMRel, ArithLogicR<"sub", GPR32Opnd>, ADD_FM_MM<0, 0x190>;
> - def SLT_MM : MMRel, SetCC_R<"slt", setlt, GPR32Opnd>, ADD_FM_MM<0, 0x350>;
> - def SLTu_MM : MMRel, SetCC_R<"sltu", setult, GPR32Opnd>,
> + def SLT_MM : MMRel, SetCC_R<"slt", setlt, GPR32Opnd, GPR32Opnd>,
> + ADD_FM_MM<0, 0x350>;
> + def SLTu_MM : MMRel, SetCC_R<"sltu", setult, GPR32Opnd, GPR32Opnd>,
> ADD_FM_MM<0, 0x390>;
> def AND_MM : MMRel, ArithLogicR<"and", GPR32Opnd, 1, II_AND, and>,
> ADD_FM_MM<0, 0x250>;
>
> Modified: llvm/trunk/lib/Target/Mips/Mips32r6InstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/Mips32r6InstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/Mips32r6InstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/Mips32r6InstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -176,7 +176,7 @@ class SDBBP_R6_ENC : SPECIAL_SDBBP_FM;
> //===----------------------------------------------------------------------===//
>
> class CMP_CONDN_DESC_BASE<string CondStr, string Typestr,
> - RegisterOperand FGROpnd,
> + RegisterOperand FGRCCOpnd, RegisterOperand FGROpnd,
> SDPatternOperator Op = null_frag> {
> dag OutOperandList = (outs FGRCCOpnd:$fd);
> dag InOperandList = (ins FGROpnd:$fs, FGROpnd:$ft);
> @@ -184,58 +184,58 @@ class CMP_CONDN_DESC_BASE<string CondStr
> list<dag> Pattern = [(set FGRCCOpnd:$fd, (Op FGROpnd:$fs, FGROpnd:$ft))];
> }
>
> -multiclass CMP_CC_M <FIELD_CMP_FORMAT Format, string Typestr,
> - RegisterOperand FGROpnd>{
> - let AdditionalPredicates = [NotInMicroMips] in {
> - def CMP_F_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_AF>,
> - CMP_CONDN_DESC_BASE<"af", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_UN_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_UN>,
> - CMP_CONDN_DESC_BASE<"un", Typestr, FGROpnd, setuo>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_EQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_EQ>,
> - CMP_CONDN_DESC_BASE<"eq", Typestr, FGROpnd, setoeq>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_UEQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_UEQ>,
> - CMP_CONDN_DESC_BASE<"ueq", Typestr, FGROpnd, setueq>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_LT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_LT>,
> - CMP_CONDN_DESC_BASE<"lt", Typestr, FGROpnd, setolt>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_ULT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_ULT>,
> - CMP_CONDN_DESC_BASE<"ult", Typestr, FGROpnd, setult>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_LE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_LE>,
> - CMP_CONDN_DESC_BASE<"le", Typestr, FGROpnd, setole>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_ULE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_ULE>,
> - CMP_CONDN_DESC_BASE<"ule", Typestr, FGROpnd, setule>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SAF_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SAF>,
> - CMP_CONDN_DESC_BASE<"saf", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SUN_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SUN>,
> - CMP_CONDN_DESC_BASE<"sun", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SEQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SEQ>,
> - CMP_CONDN_DESC_BASE<"seq", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SUEQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SUEQ>,
> - CMP_CONDN_DESC_BASE<"sueq", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SLT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SLT>,
> - CMP_CONDN_DESC_BASE<"slt", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SULT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SULT>,
> - CMP_CONDN_DESC_BASE<"sult", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SLE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SLE>,
> - CMP_CONDN_DESC_BASE<"sle", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - def CMP_SULE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SULE>,
> - CMP_CONDN_DESC_BASE<"sule", Typestr, FGROpnd>,
> - ISA_MIPS32R6, HARDFLOAT;
> - }
> +multiclass CMP_CC_M<FIELD_CMP_FORMAT Format, string Typestr,
> + RegisterOperand FGRCCOpnd, RegisterOperand FGROpnd>{
> +let AdditionalPredicates = [NotInMicroMips] in {
> +def CMP_F_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_AF>,
> + CMP_CONDN_DESC_BASE<"af", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_UN_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_UN>,
> + CMP_CONDN_DESC_BASE<"un", Typestr, FGRCCOpnd, FGROpnd,
> + setuo>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_EQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_EQ>,
> + CMP_CONDN_DESC_BASE<"eq", Typestr, FGRCCOpnd, FGROpnd,
> + setoeq>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_UEQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_UEQ>,
> + CMP_CONDN_DESC_BASE<"ueq", Typestr, FGRCCOpnd, FGROpnd,
> + setueq>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_LT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_LT>,
> + CMP_CONDN_DESC_BASE<"lt", Typestr, FGRCCOpnd, FGROpnd,
> + setolt>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_ULT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_ULT>,
> + CMP_CONDN_DESC_BASE<"ult", Typestr, FGRCCOpnd, FGROpnd,
> + setult>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_LE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_LE>,
> + CMP_CONDN_DESC_BASE<"le", Typestr, FGRCCOpnd, FGROpnd,
> + setole>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_ULE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_ULE>,
> + CMP_CONDN_DESC_BASE<"ule", Typestr, FGRCCOpnd, FGROpnd,
> + setule>, ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SAF_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SAF>,
> + CMP_CONDN_DESC_BASE<"saf", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SUN_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SUN>,
> + CMP_CONDN_DESC_BASE<"sun", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SEQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SEQ>,
> + CMP_CONDN_DESC_BASE<"seq", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SUEQ_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SUEQ>,
> + CMP_CONDN_DESC_BASE<"sueq", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SLT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SLT>,
> + CMP_CONDN_DESC_BASE<"slt", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SULT_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SULT>,
> + CMP_CONDN_DESC_BASE<"sult", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SLE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SLE>,
> + CMP_CONDN_DESC_BASE<"sle", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +def CMP_SULE_#NAME : COP1_CMP_CONDN_FM<Format, FIELD_CMP_COND_SULE>,
> + CMP_CONDN_DESC_BASE<"sule", Typestr, FGRCCOpnd, FGROpnd>,
> + ISA_MIPS32R6, HARDFLOAT;
> +}
> }
>
> //===----------------------------------------------------------------------===//
> @@ -472,7 +472,8 @@ class MUHU_DESC : MUL_R6_DESC_BASE<"mu
> class MUL_R6_DESC : MUL_R6_DESC_BASE<"mul", GPR32Opnd, mul>;
> class MULU_DESC : MUL_R6_DESC_BASE<"mulu", GPR32Opnd>;
>
> -class COP1_SEL_DESC_BASE<string instr_asm, RegisterOperand FGROpnd> {
> +class COP1_SEL_DESC_BASE<string instr_asm, RegisterOperand FGRCCOpnd,
> + RegisterOperand FGROpnd> {
> dag OutOperandList = (outs FGROpnd:$fd);
> dag InOperandList = (ins FGRCCOpnd:$fd_in, FGROpnd:$fs, FGROpnd:$ft);
> string AsmString = !strconcat(instr_asm, "\t$fd, $fs, $ft");
> @@ -482,11 +483,12 @@ class COP1_SEL_DESC_BASE<string instr_as
> string Constraints = "$fd_in = $fd";
> }
>
> -class SEL_D_DESC : COP1_SEL_DESC_BASE<"sel.d", FGR64Opnd> {
> +class SEL_D_DESC : COP1_SEL_DESC_BASE<"sel.d", FGRCC32Opnd, FGR64Opnd> {
> // We must insert a SUBREG_TO_REG around $fd_in
> bit usesCustomInserter = 1;
> }
> -class SEL_S_DESC : COP1_SEL_DESC_BASE<"sel.s", FGR32Opnd>;
> +
> +class SEL_S_DESC : COP1_SEL_DESC_BASE<"sel.s", FGRCC32Opnd, FGR32Opnd>;
>
> class SELEQNE_Z_DESC_BASE<string instr_asm, RegisterOperand GPROpnd>
> : MipsR6Arch<instr_asm> {
> @@ -693,8 +695,8 @@ let AdditionalPredicates = [NotInMicroMi
> }
> def CLO_R6 : R6MMR6Rel, CLO_R6_ENC, CLO_R6_DESC, ISA_MIPS32R6;
> def CLZ_R6 : R6MMR6Rel, CLZ_R6_ENC, CLZ_R6_DESC, ISA_MIPS32R6;
> -defm S : CMP_CC_M<FIELD_CMP_FORMAT_S, "s", FGR32Opnd>;
> -defm D : CMP_CC_M<FIELD_CMP_FORMAT_D, "d", FGR64Opnd>;
> +defm S : CMP_CC_M<FIELD_CMP_FORMAT_S, "s", FGRCC32Opnd, FGR32Opnd>;
> +defm D : CMP_CC_M<FIELD_CMP_FORMAT_D, "d", FGRCC32Opnd, FGR64Opnd>;
> def DIV : R6MMR6Rel, DIV_ENC, DIV_DESC, ISA_MIPS32R6;
> def DIVU : R6MMR6Rel, DIVU_ENC, DIVU_DESC, ISA_MIPS32R6;
> def JIALC : R6MMR6Rel, JIALC_ENC, JIALC_DESC, ISA_MIPS32R6;
>
> Modified: llvm/trunk/lib/Target/Mips/Mips64InstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/Mips64InstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/Mips64InstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/Mips64InstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -98,10 +98,14 @@ def DADDiu : ArithLogicI<"daddiu", simm
> ADDI_FM<0x19>, IsAsCheapAsAMove, ISA_MIPS3;
>
> let isCodeGenOnly = 1 in {
> -def SLTi64 : SetCC_I<"slti", setlt, simm16_64, immSExt16, GPR64Opnd>,
> - SLTI_FM<0xa>;
> -def SLTiu64 : SetCC_I<"sltiu", setult, simm16_64, immSExt16, GPR64Opnd>,
> - SLTI_FM<0xb>;
> +def SLTi64 : SetCC_I<"slti", setlt, simm16_64, immSExt16, GPR64Opnd,
> + GPR64Opnd>, SLTI_FM<0xa>;
> +def SLTiu64 : SetCC_I<"sltiu", setult, simm16_64, immSExt16, GPR64Opnd,
> + GPR64Opnd>, SLTI_FM<0xb>;
> +def SLTi64_32 : SetCC_I<"slti", setlt, simm16, immSExt16, GPR64Opnd,
> + GPR32Opnd>, SLTI_FM<0xa>;
> +def SLTiu64_32 : SetCC_I<"sltiu", setult, simm16, immSExt16, GPR64Opnd,
> + GPR32Opnd>, SLTI_FM<0xb>;
> def ANDi64 : ArithLogicI<"andi", uimm16_64, GPR64Opnd, II_AND, immZExt16, and>,
> ADDI_FM<0xc>;
> def ORi64 : ArithLogicI<"ori", uimm16_64, GPR64Opnd, II_OR, immZExt16, or>,
> @@ -122,8 +126,10 @@ def DSUB : ArithLogicR<"dsub", GPR64Op
> ISA_MIPS3;
>
> let isCodeGenOnly = 1 in {
> -def SLT64 : SetCC_R<"slt", setlt, GPR64Opnd>, ADD_FM<0, 0x2a>;
> -def SLTu64 : SetCC_R<"sltu", setult, GPR64Opnd>, ADD_FM<0, 0x2b>;
> +def SLT64 : SetCC_R<"slt", setlt, GPR64Opnd, GPR64Opnd>, ADD_FM<0, 0x2a>;
> +def SLTu64 : SetCC_R<"sltu", setult, GPR64Opnd, GPR64Opnd>, ADD_FM<0, 0x2b>;
> +def SLT64_32 : SetCC_R<"slt", setlt, GPR64Opnd, GPR32Opnd>, ADD_FM<0, 0x2a>;
> +def SLTu64_32 : SetCC_R<"sltu", setult, GPR64Opnd, GPR32Opnd>, ADD_FM<0, 0x2b>;
> def AND64 : ArithLogicR<"and", GPR64Opnd, 1, II_AND, and>, ADD_FM<0, 0x24>;
> def OR64 : ArithLogicR<"or", GPR64Opnd, 1, II_OR, or>, ADD_FM<0, 0x25>;
> def XOR64 : ArithLogicR<"xor", GPR64Opnd, 1, II_XOR, xor>, ADD_FM<0, 0x26>;
> @@ -325,8 +331,7 @@ class ExtsCins<string opstr, SDPatternOp
> class SetCC64_R<string opstr, PatFrag cond_op> :
> InstSE<(outs GPR64Opnd:$rd), (ins GPR64Opnd:$rs, GPR64Opnd:$rt),
> !strconcat(opstr, "\t$rd, $rs, $rt"),
> - [(set GPR64Opnd:$rd, (zext (cond_op GPR64Opnd:$rs,
> - GPR64Opnd:$rt)))],
> + [(set GPR64Opnd:$rd, (cond_op GPR64Opnd:$rs, GPR64Opnd:$rt))],
> II_SEQ_SNE, FrmR, opstr> {
> let TwoOperandAliasConstraint = "$rd = $rs";
> }
> @@ -334,8 +339,7 @@ class SetCC64_R<string opstr, PatFrag co
> class SetCC64_I<string opstr, PatFrag cond_op>:
> InstSE<(outs GPR64Opnd:$rt), (ins GPR64Opnd:$rs, simm10_64:$imm10),
> !strconcat(opstr, "\t$rt, $rs, $imm10"),
> - [(set GPR64Opnd:$rt, (zext (cond_op GPR64Opnd:$rs,
> - immSExt10_64:$imm10)))],
> + [(set GPR64Opnd:$rt, (cond_op GPR64Opnd:$rs, immSExt10_64:$imm10))],
> II_SEQI_SNEI, FrmI, opstr> {
> let TwoOperandAliasConstraint = "$rt = $rs";
> }
> @@ -344,7 +348,7 @@ class CBranchBitNum<string opstr, DAGOpe
> RegisterOperand RO, Operand ImmOp, bits<64> shift = 1> :
> InstSE<(outs), (ins RO:$rs, ImmOp:$p, opnd:$offset),
> !strconcat(opstr, "\t$rs, $p, $offset"),
> - [(brcond (i32 (cond_op (and RO:$rs, (shl shift, immZExt5_64:$p)), 0)),
> + [(brcond (i64 (cond_op (and RO:$rs, (shl shift, immZExt5_64:$p)), 0)),
> bb:$offset)], II_BBIT, FrmI, opstr> {
> let isBranch = 1;
> let isTerminator = 1;
> @@ -480,20 +484,30 @@ def : WrapperPat<tblockaddress, DADDiu,
> def : WrapperPat<tjumptable, DADDiu, GPR64>;
> def : WrapperPat<tglobaltlsaddr, DADDiu, GPR64>;
>
> -defm : BrcondPats<GPR64, BEQ64, BNE64, SLT64, SLTu64, SLTi64, SLTiu64,
> - ZERO_64>;
> +defm : BrcondPats1<GPR64, BEQ64, BNE64, ZERO_64, i64>;
> +defm : BrcondPats1<GPR32, BEQ, BNE, ZERO, i64>;
>
> -def : MipsPat<(brcond (i32 (setlt i64:$lhs, 1)), bb:$dst),
> +def : MipsPat<(brcond (i64 (setlt i64:$lhs, 1)), bb:$dst),
> (BLEZ64 i64:$lhs, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setgt i64:$lhs, -1)), bb:$dst),
> +def : MipsPat<(brcond (i64 (setgt i64:$lhs, -1)), bb:$dst),
> (BGEZ64 i64:$lhs, bb:$dst)>;
> +def : MipsPat<(brcond (i64 (setlt i32:$lhs, 1)), bb:$dst),
> + (BLEZ64 (SUBREG_TO_REG (i32 0), i32:$lhs, sub_32), bb:$dst)>;
> +def : MipsPat<(brcond (i64 (setgt i32:$lhs, -1)), bb:$dst),
> + (BGEZ64 (SUBREG_TO_REG (i32 0), i32:$lhs, sub_32), bb:$dst)>;
>
> // setcc patterns
> defm : SeteqPats<GPR64, SLTiu64, XOR64, SLTu64, ZERO_64>;
> -defm : SetlePats<GPR64, SLT64, SLTu64>;
> +defm : SetlePats<GPR64, SLT64, SLTu64, XORi64>;
> defm : SetgtPats<GPR64, SLT64, SLTu64>;
> -defm : SetgePats<GPR64, SLT64, SLTu64>;
> -defm : SetgeImmPats<GPR64, SLTi64, SLTiu64>;
> +defm : SetgePats<GPR64, SLT64, SLTu64, XORi64>;
> +defm : SetgeImmPats<GPR64, SLTi64, SLTiu64, XORi64>;
> +
> +defm : SeteqPats<GPR32, SLTiu64_32, XOR, SLTu64_32, ZERO>;
> +defm : SetlePats<GPR32, SLT64_32, SLTu64_32, XORi64>;
> +defm : SetgtPats<GPR32, SLT64_32, SLTu64_32>;
> +defm : SetgePats<GPR32, SLT64_32, SLTu64_32, XORi64>;
> +defm : SetgeImmPats<GPR32, SLTi64_32, SLTiu64_32, XORi64>;
>
> // truncate
> def : MipsPat<(trunc (assertsext GPR64:$src)),
> @@ -538,13 +552,13 @@ let AdditionalPredicates = [NotDSP] in {
>
> // Octeon bbit0/bbit1 MipsPattern
> let Predicates = [HasMips64, HasCnMips] in {
> -def : MipsPat<(brcond (i32 (seteq (and i64:$lhs, PowerOf2LO:$mask), 0)), bb:$dst),
> +def : MipsPat<(brcond (i64 (seteq (and i64:$lhs, PowerOf2LO:$mask), 0)), bb:$dst),
> (BBIT0 i64:$lhs, (Log2LO PowerOf2LO:$mask), bb:$dst)>;
> -def : MipsPat<(brcond (i32 (seteq (and i64:$lhs, PowerOf2HI:$mask), 0)), bb:$dst),
> +def : MipsPat<(brcond (i64 (seteq (and i64:$lhs, PowerOf2HI:$mask), 0)), bb:$dst),
> (BBIT032 i64:$lhs, (Log2HI PowerOf2HI:$mask), bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setne (and i64:$lhs, PowerOf2LO:$mask), 0)), bb:$dst),
> +def : MipsPat<(brcond (i64 (setne (and i64:$lhs, PowerOf2LO:$mask), 0)), bb:$dst),
> (BBIT1 i64:$lhs, (Log2LO PowerOf2LO:$mask), bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setne (and i64:$lhs, PowerOf2HI:$mask), 0)), bb:$dst),
> +def : MipsPat<(brcond (i64 (setne (and i64:$lhs, PowerOf2HI:$mask), 0)), bb:$dst),
> (BBIT132 i64:$lhs, (Log2HI PowerOf2HI:$mask), bb:$dst)>;
> }
>
>
> Modified: llvm/trunk/lib/Target/Mips/Mips64r6InstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/Mips64r6InstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/Mips64r6InstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/Mips64r6InstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -74,6 +74,11 @@ class LLD_R6_DESC : LL_R6_DESC_BASE<"l
> class SCD_R6_DESC : SC_R6_DESC_BASE<"scd", GPR64Opnd>;
> class SELEQZ64_DESC : SELEQNE_Z_DESC_BASE<"seleqz", GPR64Opnd>;
> class SELNEZ64_DESC : SELEQNE_Z_DESC_BASE<"selnez", GPR64Opnd>;
> +class SEL64_S_DESC : COP1_SEL_DESC_BASE<"sel.s", FGRCC64Opnd, FGR32Opnd> {
> + // We must copy FGRCC64Opnd's subreg into FGR32.
> + bit usesCustomInserter = 1;
> +}
> +class SEL64_D_DESC : COP1_SEL_DESC_BASE<"sel.d", FGRCC64Opnd, FGR64Opnd>;
>
> //===----------------------------------------------------------------------===//
> //
> @@ -100,11 +105,18 @@ def DMUHU: DMUHU_ENC, DMUHU_DESC, ISA_MI
> def DMUL_R6: DMUL_R6_ENC, DMUL_R6_DESC, ISA_MIPS64R6;
> def DMULU: DMULU_ENC, DMULU_DESC, ISA_MIPS64R6;
> def LDPC: LDPC_ENC, LDPC_DESC, ISA_MIPS64R6;
> -def LLD_R6 : LLD_R6_ENC, LLD_R6_DESC, ISA_MIPS32R6;
> -def SCD_R6 : SCD_R6_ENC, SCD_R6_DESC, ISA_MIPS32R6;
> -let DecoderNamespace = "Mips32r6_64r6_GP64" in {
> - def SELEQZ64 : SELEQZ_ENC, SELEQZ64_DESC, ISA_MIPS32R6, GPR_64;
> - def SELNEZ64 : SELNEZ_ENC, SELNEZ64_DESC, ISA_MIPS32R6, GPR_64;
> +def LLD_R6 : LLD_R6_ENC, LLD_R6_DESC, ISA_MIPS64R6;
> +def SCD_R6 : SCD_R6_ENC, SCD_R6_DESC, ISA_MIPS64R6;
> +let AdditionalPredicates = [NotInMicroMips],
> + DecoderNamespace = "Mips64r6" in {
> + def SELEQZ64 : SELEQZ_ENC, SELEQZ64_DESC, ISA_MIPS64R6;
> + def SELNEZ64 : SELNEZ_ENC, SELNEZ64_DESC, ISA_MIPS64R6;
> + def SEL64_S : SEL_S_ENC, SEL64_S_DESC, ISA_MIPS64R6, HARDFLOAT;
> + def SEL64_D : SEL_D_ENC, SEL64_D_DESC, ISA_MIPS64R6, HARDFLOAT;
> + defm S64: CMP_CC_M<FIELD_CMP_FORMAT_S, "s", FGRCC64Opnd, FGR32Opnd>,
> + ISA_MIPS64R6;
> + defm D64: CMP_CC_M<FIELD_CMP_FORMAT_D, "d", FGRCC64Opnd, FGR64Opnd>,
> + ISA_MIPS64R6;
> }
>
> //===----------------------------------------------------------------------===//
> @@ -121,99 +133,71 @@ def : MipsInstAlias<"jr $rs", (JALR64 ZE
> //
> //===----------------------------------------------------------------------===//
>
> +// comparisons supported via another comparison
> +defm S64: Cmp_Pats<f32, NOR64, ZERO_64>, ISA_MIPS64R6;
> +defm D64: Cmp_Pats<f64, NOR64, ZERO_64>, ISA_MIPS64R6;
> +
> +// i32 selects
> +defm : SelectInt_Pats<i32, OR, XORi, SLTi, SLTiu, SELEQZ, SELNEZ,
> + immZExt16, i64>, ISA_MIPS64R6;
> +
> +def : MipsPat<(select i64:$cond, i32:$t, i32:$f),
> + (OR (SELNEZ i32:$t, (EXTRACT_SUBREG i64:$cond, sub_32)),
> + (SELEQZ i32:$f, (EXTRACT_SUBREG i64:$cond, sub_32)))>,
> + ISA_MIPS64R6;
> +def : MipsPat<(select i64:$cond, i32:$t, immz),
> + (SELNEZ i32:$t, (EXTRACT_SUBREG i64:$cond, sub_32))>,
> + ISA_MIPS64R6;
> +def : MipsPat<(select i64:$cond, immz, i32:$f),
> + (SELEQZ i32:$f, (EXTRACT_SUBREG i64:$cond, sub_32))>,
> + ISA_MIPS64R6;
> +
> // i64 selects
> +defm : SelectInt_Pats<i64, OR64, XORi64, SLTi64, SLTiu64, SELEQZ64, SELNEZ64,
> + immZExt16_64, i64>, ISA_MIPS64R6;
> +
> def : MipsPat<(select i64:$cond, i64:$t, i64:$f),
> (OR64 (SELNEZ64 i64:$t, i64:$cond),
> (SELEQZ64 i64:$f, i64:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i64:$cond, immz)), i64:$t, i64:$f),
> - (OR64 (SELEQZ64 i64:$t, i64:$cond),
> - (SELNEZ64 i64:$f, i64:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i64:$cond, immz)), i64:$t, i64:$f),
> - (OR64 (SELNEZ64 i64:$t, i64:$cond),
> - (SELEQZ64 i64:$f, i64:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i64:$cond, immZExt16_64:$imm)), i64:$t, i64:$f),
> - (OR64 (SELEQZ64 i64:$t, (XORi64 i64:$cond, immZExt16_64:$imm)),
> - (SELNEZ64 i64:$f, (XORi64 i64:$cond, immZExt16_64:$imm)))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i64:$cond, immZExt16_64:$imm)), i64:$t, i64:$f),
> - (OR64 (SELNEZ64 i64:$t, (XORi64 i64:$cond, immZExt16_64:$imm)),
> - (SELEQZ64 i64:$f, (XORi64 i64:$cond, immZExt16_64:$imm)))>,
> - ISA_MIPS64R6;
> -def : MipsPat<
> - (select (i32 (setgt i64:$cond, immSExt16Plus1:$imm)), i64:$t, i64:$f),
> - (OR64 (SELEQZ64 i64:$t,
> - (SUBREG_TO_REG (i64 0), (SLTi64 i64:$cond, (Plus1 imm:$imm)),
> - sub_32)),
> - (SELNEZ64 i64:$f,
> - (SUBREG_TO_REG (i64 0), (SLTi64 i64:$cond, (Plus1 imm:$imm)),
> - sub_32)))>,
> - ISA_MIPS64R6;
> -def : MipsPat<
> - (select (i32 (setugt i64:$cond, immSExt16Plus1:$imm)), i64:$t, i64:$f),
> - (OR64 (SELEQZ64 i64:$t,
> - (SUBREG_TO_REG (i64 0), (SLTiu64 i64:$cond, (Plus1 imm:$imm)),
> - sub_32)),
> - (SELNEZ64 i64:$f,
> - (SUBREG_TO_REG (i64 0), (SLTiu64 i64:$cond, (Plus1 imm:$imm)),
> - sub_32)))>,
> - ISA_MIPS64R6;
> -
> -def : MipsPat<(select (i32 (setne i64:$cond, immz)), i64:$t, immz),
> - (SELNEZ64 i64:$t, i64:$cond)>, ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i64:$cond, immz)), i64:$t, immz),
> - (SELEQZ64 i64:$t, i64:$cond)>, ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i64:$cond, immz)), immz, i64:$f),
> - (SELEQZ64 i64:$f, i64:$cond)>, ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i64:$cond, immz)), immz, i64:$f),
> - (SELNEZ64 i64:$f, i64:$cond)>, ISA_MIPS64R6;
> + ISA_MIPS64R6;
>
> // i64 selects from an i32 comparison
> -// One complicating factor here is that bits 32-63 of an i32 are undefined.
> -// FIXME: Ideally, setcc would always produce an i64 on MIPS64 targets.
> -// This would allow us to remove the sign-extensions here.
> def : MipsPat<(select i32:$cond, i64:$t, i64:$f),
> - (OR64 (SELNEZ64 i64:$t, (SLL64_32 i32:$cond)),
> - (SELEQZ64 i64:$f, (SLL64_32 i32:$cond)))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i32:$cond, immz)), i64:$t, i64:$f),
> - (OR64 (SELEQZ64 i64:$t, (SLL64_32 i32:$cond)),
> - (SELNEZ64 i64:$f, (SLL64_32 i32:$cond)))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i32:$cond, immz)), i64:$t, i64:$f),
> - (OR64 (SELNEZ64 i64:$t, (SLL64_32 i32:$cond)),
> - (SELEQZ64 i64:$f, (SLL64_32 i32:$cond)))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i32:$cond, immZExt16:$imm)), i64:$t, i64:$f),
> - (OR64 (SELEQZ64 i64:$t, (SLL64_32 (XORi i32:$cond,
> - immZExt16:$imm))),
> - (SELNEZ64 i64:$f, (SLL64_32 (XORi i32:$cond,
> - immZExt16:$imm))))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i32:$cond, immZExt16:$imm)), i64:$t, i64:$f),
> - (OR64 (SELNEZ64 i64:$t, (SLL64_32 (XORi i32:$cond,
> - immZExt16:$imm))),
> - (SELEQZ64 i64:$f, (SLL64_32 (XORi i32:$cond,
> - immZExt16:$imm))))>,
> - ISA_MIPS64R6;
> + (OR64 (SELNEZ64 i64:$t, (SLL64_32 i32:$cond)),
> + (SELEQZ64 i64:$f, (SLL64_32 i32:$cond)))>,
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (seteq i32:$cond, immz)), i64:$t, i64:$f),
> + (OR64 (SELEQZ64 i64:$t, (SLL64_32 i32:$cond)),
> + (SELNEZ64 i64:$f, (SLL64_32 i32:$cond)))>,
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (setne i32:$cond, immz)), i64:$t, i64:$f),
> + (OR64 (SELNEZ64 i64:$t, (SLL64_32 i32:$cond)),
> + (SELEQZ64 i64:$f, (SLL64_32 i32:$cond)))>,
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (seteq i32:$cond, immZExt16:$imm)), i64:$t, i64:$f),
> + (OR64 (SELEQZ64 i64:$t, (SLL64_32 (XORi i32:$cond, immZExt16:$imm))),
> + (SELNEZ64 i64:$f, (SLL64_32 (XORi i32:$cond, immZExt16:$imm))))>,
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (setne i32:$cond, immZExt16:$imm)), i64:$t, i64:$f),
> + (OR64 (SELNEZ64 i64:$t, (SLL64_32 (XORi i32:$cond, immZExt16:$imm))),
> + (SELEQZ64 i64:$f, (SLL64_32 (XORi i32:$cond, immZExt16:$imm))))>,
> + ISA_MIPS64R6;
>
> def : MipsPat<(select i32:$cond, i64:$t, immz),
> (SELNEZ64 i64:$t, (SLL64_32 i32:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i32:$cond, immz)), i64:$t, immz),
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (setne i32:$cond, immz)), i64:$t, immz),
> (SELNEZ64 i64:$t, (SLL64_32 i32:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i32:$cond, immz)), i64:$t, immz),
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (seteq i32:$cond, immz)), i64:$t, immz),
> (SELEQZ64 i64:$t, (SLL64_32 i32:$cond))>,
> - ISA_MIPS64R6;
> + ISA_MIPS64R6;
> def : MipsPat<(select i32:$cond, immz, i64:$f),
> (SELEQZ64 i64:$f, (SLL64_32 i32:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (setne i32:$cond, immz)), immz, i64:$f),
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (setne i32:$cond, immz)), immz, i64:$f),
> (SELEQZ64 i64:$f, (SLL64_32 i32:$cond))>,
> - ISA_MIPS64R6;
> -def : MipsPat<(select (i32 (seteq i32:$cond, immz)), immz, i64:$f),
> + ISA_MIPS64R6;
> +def : MipsPat<(select (i64 (seteq i32:$cond, immz)), immz, i64:$f),
> (SELNEZ64 i64:$f, (SLL64_32 i32:$cond))>,
> - ISA_MIPS64R6;
> + ISA_MIPS64R6;
>
> Modified: llvm/trunk/lib/Target/Mips/MipsCondMov.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsCondMov.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsCondMov.td (original)
> +++ llvm/trunk/lib/Target/Mips/MipsCondMov.td Tue Mar 1 04:08:01 2016
> @@ -52,57 +52,6 @@ class CMov_F_F_FT<string opstr, Register
> let Constraints = "$F = $fd";
> }
>
> -// select patterns
> -multiclass MovzPats0<RegisterClass CRC, RegisterClass DRC,
> - Instruction MOVZInst, Instruction SLTOp,
> - Instruction SLTuOp, Instruction SLTiOp,
> - Instruction SLTiuOp> {
> - def : MipsPat<(select (i32 (setge CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTOp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> - def : MipsPat<(select (i32 (setuge CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTuOp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> - def : MipsPat<(select (i32 (setge CRC:$lhs, immSExt16:$rhs)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTiOp CRC:$lhs, immSExt16:$rhs), DRC:$F)>;
> - def : MipsPat<(select (i32 (setuge CRC:$lh, immSExt16:$rh)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTiuOp CRC:$lh, immSExt16:$rh), DRC:$F)>;
> - def : MipsPat<(select (i32 (setle CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTOp CRC:$rhs, CRC:$lhs), DRC:$F)>;
> - def : MipsPat<(select (i32 (setule CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTuOp CRC:$rhs, CRC:$lhs), DRC:$F)>;
> - def : MipsPat<(select (i32 (setgt CRC:$lhs, immSExt16Plus1:$rhs)),
> - DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTiOp CRC:$lhs, (Plus1 imm:$rhs)), DRC:$F)>;
> - def : MipsPat<(select (i32 (setugt CRC:$lhs, immSExt16Plus1:$rhs)),
> - DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (SLTiuOp CRC:$lhs, (Plus1 imm:$rhs)),
> - DRC:$F)>;
> -}
> -
> -multiclass MovzPats1<RegisterClass CRC, RegisterClass DRC,
> - Instruction MOVZInst, Instruction XOROp> {
> - def : MipsPat<(select (i32 (seteq CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (XOROp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> - def : MipsPat<(select (i32 (seteq CRC:$lhs, 0)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, CRC:$lhs, DRC:$F)>;
> -}
> -
> -multiclass MovzPats2<RegisterClass CRC, RegisterClass DRC,
> - Instruction MOVZInst, Instruction XORiOp> {
> - def : MipsPat<
> - (select (i32 (seteq CRC:$lhs, immZExt16:$uimm16)), DRC:$T, DRC:$F),
> - (MOVZInst DRC:$T, (XORiOp CRC:$lhs, immZExt16:$uimm16), DRC:$F)>;
> -}
> -
> -multiclass MovnPats<RegisterClass CRC, RegisterClass DRC, Instruction MOVNInst,
> - Instruction XOROp> {
> - def : MipsPat<(select (i32 (setne CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> - (MOVNInst DRC:$T, (XOROp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> - def : MipsPat<(select CRC:$cond, DRC:$T, DRC:$F),
> - (MOVNInst DRC:$T, CRC:$cond, DRC:$F)>;
> - def : MipsPat<(select (i32 (setne CRC:$lhs, 0)),DRC:$T, DRC:$F),
> - (MOVNInst DRC:$T, CRC:$lhs, DRC:$F)>;
> -}
> -
> // Instantiation of instructions.
> def MOVZ_I_I : MMRel, CMov_I_I_FT<"movz", GPR32Opnd, GPR32Opnd, II_MOVZ>,
> ADD_FM<0, 0xa>, INSN_MIPS4_32_NOT_32R6_64R6;
> @@ -199,96 +148,189 @@ let DecoderNamespace = "Mips64" in {
> CMov_F_F_FM<17, 0>, INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> }
>
> -// Instantiation of conditional move patterns.
> -defm : MovzPats0<GPR32, GPR32, MOVZ_I_I, SLT, SLTu, SLTi, SLTiu>,
> +// select patterns
> +multiclass MovzPats0<RegisterClass CRC, RegisterClass DRC,
> + Instruction MOVZInst, Instruction SLTOp,
> + Instruction SLTuOp, Instruction SLTiOp,
> + Instruction SLTiuOp, ValueType VT> {
> +// reg, reg
> +def : MipsPat<(select (VT (setge CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTOp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> +def : MipsPat<(select (VT (setuge CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTuOp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> +def : MipsPat<(select (VT (setle CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTOp CRC:$rhs, CRC:$lhs), DRC:$F)>;
> +def : MipsPat<(select (VT (setule CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTuOp CRC:$rhs, CRC:$lhs), DRC:$F)>;
> +
> +// reg, imm
> +def : MipsPat<(select (VT (setge CRC:$lhs, immSExt16:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTiOp CRC:$lhs, immSExt16:$rhs), DRC:$F)>;
> +def : MipsPat<(select (VT (setuge CRC:$lh, immSExt16:$rh)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTiuOp CRC:$lh, immSExt16:$rh), DRC:$F)>;
> +def : MipsPat<
> + (select (VT (setgt CRC:$lhs, immSExt16Plus1:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTiOp CRC:$lhs, (Plus1 imm:$rhs)), DRC:$F)>;
> +def : MipsPat<
> + (select (VT (setugt CRC:$lhs, immSExt16Plus1:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (SLTiuOp CRC:$lhs, (Plus1 imm:$rhs)), DRC:$F)>;
> +}
> +
> +// We have to wrap the instantiation of the MovzPats0 patterns in another
> +// multiclass that specifies the result type of the SETCC nodes. The patterns
> +// with VT=i64 (or i32) will be ignored when GPR-width=i32 (or i64).
> +multiclass MovzPats0_SuperClass<ValueType VT> {
> +defm : MovzPats0<GPR32, GPR32, MOVZ_I_I, SLT, SLTu, SLTi, SLTiu, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6;
> -defm : MovzPats1<GPR32, GPR32, MOVZ_I_I, XOR>, INSN_MIPS4_32_NOT_32R6_64R6;
> -defm : MovzPats2<GPR32, GPR32, MOVZ_I_I, XORi>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovzPats0<GPR32, GPR64, MOVZ_I_I64, SLT, SLTu, SLTi, SLTiu, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovzPats0<GPR32, FGR32, MOVZ_I_S, SLT, SLTu, SLTi, SLTiu, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovzPats0<GPR32, AFGR64, MOVZ_I_D32, SLT, SLTu, SLTi, SLTiu, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, FGR_32;
> +defm : MovzPats0<GPR32, FGR64, MOVZ_I_D64, SLT, SLTu, SLTi, SLTiu, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> +
> +defm : MovzPats0<GPR32, GPR32, MOVZ_I64_I, SLT64_32, SLTu64_32, SLTi64_32,
> + SLTiu64_32, VT>, INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovzPats0<GPR32, GPR64, MOVZ_I64_I64, SLT64_32, SLTu64_32, SLTi64_32,
> + SLTiu64_32, VT>, INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovzPats0<GPR32, FGR32, MOVZ_I64_S, SLT64_32, SLTu64_32, SLTi64_32,
> + SLTiu64_32, VT>, INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovzPats0<GPR32, FGR64, MOVZ_I64_D64, SLT64_32, SLTu64_32, SLTi64_32,
> + SLTiu64_32, VT>, INSN_MIPS4_32_NOT_32R6_64R6, GPR_64, FGR_64;
>
> -defm : MovzPats0<GPR32, GPR64, MOVZ_I_I64, SLT, SLTu, SLTi, SLTiu>,
> +defm : MovzPats0<GPR64, GPR32, MOVZ_I64_I, SLT64, SLTu64, SLTi64, SLTiu64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats0<GPR64, GPR32, MOVZ_I_I, SLT64, SLTu64, SLTi64, SLTiu64>,
> +defm : MovzPats0<GPR64, GPR64, MOVZ_I64_I64, SLT64, SLTu64, SLTi64, SLTiu64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats0<GPR64, GPR64, MOVZ_I_I64, SLT64, SLTu64, SLTi64, SLTiu64>,
> +defm : MovzPats0<GPR64, FGR32, MOVZ_I64_S, SLT64, SLTu64, SLTi64, SLTiu64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats1<GPR32, GPR64, MOVZ_I_I64, XOR>,
> +defm : MovzPats0<GPR64, FGR64, MOVZ_I64_D64, SLT64, SLTu64, SLTi64, SLTiu64, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64, FGR_64;
> +}
> +
> +defm : MovzPats0_SuperClass<i32>;
> +defm : MovzPats0_SuperClass<i64>;
> +
> +multiclass MovzPats1<RegisterClass CRC, RegisterClass DRC,
> + Instruction MOVZInst, Instruction XOROp, ValueType VT> {
> +def : MipsPat<(select (VT (seteq CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (XOROp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> +def : MipsPat<(select (VT (seteq CRC:$lhs, 0)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, CRC:$lhs, DRC:$F)>;
> +}
> +
> +multiclass MovzPats1_SuperClass<ValueType VT> {
> +defm : MovzPats1<GPR32, GPR32, MOVZ_I_I, XOR, VT>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovzPats1<GPR32, GPR64, MOVZ_I_I64, XOR, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovzPats1<GPR32, FGR32, MOVZ_I_S, XOR, VT>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovzPats1<GPR32, AFGR64, MOVZ_I_D32, XOR, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, FGR_32;
> +defm : MovzPats1<GPR32, FGR64, MOVZ_I_D64, XOR, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> +
> +defm : MovzPats1<GPR64, GPR32, MOVZ_I64_I, XOR64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats1<GPR64, GPR32, MOVZ_I64_I, XOR64>,
> +defm : MovzPats1<GPR64, GPR64, MOVZ_I64_I64, XOR64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats1<GPR64, GPR64, MOVZ_I64_I64, XOR64>,
> +defm : MovzPats1<GPR64, FGR32, MOVZ_I64_S, XOR64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats2<GPR32, GPR64, MOVZ_I_I64, XORi>,
> +defm : MovzPats1<GPR64, FGR64, MOVZ_I64_D64, XOR64, VT>,
> + INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> +}
> +
> +defm : MovzPats1_SuperClass<i32>;
> +defm : MovzPats1_SuperClass<i64>;
> +
> +multiclass MovzPats2<RegisterClass CRC, RegisterClass DRC,
> + Instruction MOVZInst, Instruction XORiOp, ValueType VT> {
> +def : MipsPat<
> + (select (VT (seteq CRC:$lhs, immZExt16:$uimm16)), DRC:$T, DRC:$F),
> + (MOVZInst DRC:$T, (XORiOp CRC:$lhs, immZExt16:$uimm16), DRC:$F)>;
> +}
> +
> +multiclass MovzPats2_SuperClass<ValueType VT> {
> +defm : MovzPats2<GPR32, GPR32, MOVZ_I_I, XORi, VT>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovzPats2<GPR32, GPR64, MOVZ_I_I64, XORi, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats2<GPR64, GPR32, MOVZ_I64_I, XORi64>,
> +defm : MovzPats2<GPR64, GPR32, MOVZ_I64_I, XORi64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats2<GPR64, GPR64, MOVZ_I64_I64, XORi64>,
> +defm : MovzPats2<GPR64, GPR64, MOVZ_I64_I64, XORi64, VT>,
> INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +}
>
> -defm : MovnPats<GPR32, GPR32, MOVN_I_I, XOR>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovzPats2_SuperClass<i32>;
> +defm : MovzPats2_SuperClass<i64>;
>
> -defm : MovnPats<GPR32, GPR64, MOVN_I_I64, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> - GPR_64;
> -defm : MovnPats<GPR64, GPR32, MOVN_I64_I, XOR64>, INSN_MIPS4_32_NOT_32R6_64R6,
> - GPR_64;
> -defm : MovnPats<GPR64, GPR64, MOVN_I64_I64, XOR64>, INSN_MIPS4_32_NOT_32R6_64R6,
> - GPR_64;
> +multiclass MovnPats<RegisterClass CRC, RegisterClass DRC, Instruction MOVNInst,
> + Instruction XOROp, ValueType VT = i32> {
> + def : MipsPat<(select (VT (setne CRC:$lhs, CRC:$rhs)), DRC:$T, DRC:$F),
> + (MOVNInst DRC:$T, (XOROp CRC:$lhs, CRC:$rhs), DRC:$F)>;
> + def : MipsPat<(select CRC:$cond, DRC:$T, DRC:$F),
> + (MOVNInst DRC:$T, CRC:$cond, DRC:$F)>;
> + def : MipsPat<(select (VT (setne CRC:$lhs, 0)),DRC:$T, DRC:$F),
> + (MOVNInst DRC:$T, CRC:$lhs, DRC:$F)>;
> +}
>
> -defm : MovzPats0<GPR32, FGR32, MOVZ_I_S, SLT, SLTu, SLTi, SLTiu>,
> - INSN_MIPS4_32_NOT_32R6_64R6;
> -defm : MovzPats1<GPR32, FGR32, MOVZ_I_S, XOR>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovnPats<GPR32, GPR32, MOVN_I_I, XOR>, INSN_MIPS4_32_NOT_32R6_64R6;
> defm : MovnPats<GPR32, FGR32, MOVN_I_S, XOR>, INSN_MIPS4_32_NOT_32R6_64R6;
> +defm : MovnPats<GPR32, AFGR64, MOVN_I_D32, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> + FGR_32;
> +defm : MovnPats<GPR32, FGR64, MOVN_I_D64, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> + FGR_64;
>
> -defm : MovzPats0<GPR64, FGR32, MOVZ_I_S, SLT64, SLTu64, SLTi64, SLTiu64>,
> - INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> -defm : MovzPats1<GPR64, FGR32, MOVZ_I64_S, XOR64>, INSN_MIPS4_32_NOT_32R6_64R6,
> +defm : MovnPats<GPR32, GPR32, MOVN_I_I, XOR, i64>, INSN_MIPS4_32_NOT_32R6_64R6,
> GPR_64;
> -defm : MovnPats<GPR64, FGR32, MOVN_I64_S, XOR64>, INSN_MIPS4_32_NOT_32R6_64R6,
> +defm : MovnPats<GPR32, GPR64, MOVN_I_I64, XOR, i64>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovnPats<GPR32, FGR32, MOVN_I_S, XOR, i64>, INSN_MIPS4_32_NOT_32R6_64R6,
> GPR_64;
> -
> -defm : MovzPats0<GPR32, AFGR64, MOVZ_I_D32, SLT, SLTu, SLTi, SLTiu>,
> +defm : MovnPats<GPR32, AFGR64, MOVN_I_D32, XOR, i64>,
> INSN_MIPS4_32_NOT_32R6_64R6, FGR_32;
> -defm : MovzPats1<GPR32, AFGR64, MOVZ_I_D32, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> - FGR_32;
> -defm : MovnPats<GPR32, AFGR64, MOVN_I_D32, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> - FGR_32;
> +defm : MovnPats<GPR32, FGR64, MOVN_I_D64, XOR, i64>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64, FGR_64;
>
> -defm : MovzPats0<GPR32, FGR64, MOVZ_I_D64, SLT, SLTu, SLTi, SLTiu>,
> - INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> -defm : MovzPats0<GPR64, FGR64, MOVZ_I_D64, SLT64, SLTu64, SLTi64, SLTiu64>,
> - INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> -defm : MovzPats1<GPR32, FGR64, MOVZ_I_D64, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> - FGR_64;
> -defm : MovzPats1<GPR64, FGR64, MOVZ_I64_D64, XOR64>,
> - INSN_MIPS4_32_NOT_32R6_64R6, FGR_64;
> -defm : MovnPats<GPR32, FGR64, MOVN_I_D64, XOR>, INSN_MIPS4_32_NOT_32R6_64R6,
> - FGR_64;
> -defm : MovnPats<GPR64, FGR64, MOVN_I64_D64, XOR64>, INSN_MIPS4_32_NOT_32R6_64R6,
> - FGR_64;
> +defm : MovnPats<GPR64, GPR32, MOVN_I64_I, XOR64, i64>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovnPats<GPR64, GPR64, MOVN_I64_I64, XOR64, i64>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovnPats<GPR64, FGR32, MOVN_I64_S, XOR64, i64>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64;
> +defm : MovnPats<GPR64, FGR64, MOVN_I64_D64, XOR64, i64>,
> + INSN_MIPS4_32_NOT_32R6_64R6, GPR_64, FGR_64;
>
> // For targets that don't have conditional-move instructions
> // we have to match SELECT nodes with pseudo instructions.
> let usesCustomInserter = 1 in {
> - class Select_Pseudo<RegisterOperand RC> :
> - PseudoSE<(outs RC:$dst), (ins GPR32Opnd:$cond, RC:$T, RC:$F),
> - [(set RC:$dst, (select GPR32Opnd:$cond, RC:$T, RC:$F))]>,
> + class Select_Pseudo<RegisterOperand RC, RegisterOperand RO> :
> + PseudoSE<(outs RO:$dst), (ins RC:$cond, RO:$T, RO:$F),
> + [(set RO:$dst, (select RC:$cond, RO:$T, RO:$F))]>,
> ISA_MIPS1_NOT_4_32;
>
> - class SelectFP_Pseudo_T<RegisterOperand RC> :
> - PseudoSE<(outs RC:$dst), (ins GPR32Opnd:$cond, RC:$T, RC:$F),
> - [(set RC:$dst, (MipsCMovFP_T RC:$T, GPR32Opnd:$cond, RC:$F))]>,
> + class SelectFP_Pseudo_T<RegisterOperand RO> :
> + PseudoSE<(outs RO:$dst), (ins GPR32Opnd:$cond, RO:$T, RO:$F),
> + [(set RO:$dst, (MipsCMovFP_T RO:$T, GPR32Opnd:$cond, RO:$F))]>,
> ISA_MIPS1_NOT_4_32;
>
> - class SelectFP_Pseudo_F<RegisterOperand RC> :
> - PseudoSE<(outs RC:$dst), (ins GPR32Opnd:$cond, RC:$T, RC:$F),
> - [(set RC:$dst, (MipsCMovFP_F RC:$T, GPR32Opnd:$cond, RC:$F))]>,
> + class SelectFP_Pseudo_F<RegisterOperand RO> :
> + PseudoSE<(outs RO:$dst), (ins GPR32Opnd:$cond, RO:$T, RO:$F),
> + [(set RO:$dst, (MipsCMovFP_F RO:$T, GPR32Opnd:$cond, RO:$F))]>,
> ISA_MIPS1_NOT_4_32;
> }
>
> -def PseudoSELECT_I : Select_Pseudo<GPR32Opnd>;
> -def PseudoSELECT_I64 : Select_Pseudo<GPR64Opnd>;
> -def PseudoSELECT_S : Select_Pseudo<FGR32Opnd>;
> -def PseudoSELECT_D32 : Select_Pseudo<AFGR64Opnd>, FGR_32;
> -def PseudoSELECT_D64 : Select_Pseudo<FGR64Opnd>, FGR_64;
> +def PseudoSELECT_I : Select_Pseudo<GPR32Opnd, GPR32Opnd>;
> +def PseudoSELECT_I64 : Select_Pseudo<GPR32Opnd, GPR64Opnd>;
> +def PseudoSELECT_S : Select_Pseudo<GPR32Opnd, FGR32Opnd>;
> +def PseudoSELECT_D32 : Select_Pseudo<GPR32Opnd, AFGR64Opnd>, FGR_32;
> +def PseudoSELECT_D64 : Select_Pseudo<GPR32Opnd, FGR64Opnd>, FGR_64;
> +
> +def PseudoSELECT64_I : Select_Pseudo<GPR64Opnd, GPR32Opnd>, GPR_64;
> +def PseudoSELECT64_I64 : Select_Pseudo<GPR64Opnd, GPR64Opnd>, GPR_64;
> +def PseudoSELECT64_S : Select_Pseudo<GPR64Opnd, FGR32Opnd>, GPR_64;
> +def PseudoSELECT64_D32 : Select_Pseudo<GPR64Opnd, AFGR64Opnd>, GPR_64, FGR_32;
> +def PseudoSELECT64_D64 : Select_Pseudo<GPR64Opnd, FGR64Opnd>, GPR_64, FGR_64;
>
> def PseudoSELECTFP_T_I : SelectFP_Pseudo_T<GPR32Opnd>;
> def PseudoSELECTFP_T_I64 : SelectFP_Pseudo_T<GPR64Opnd>;
>
> Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp (original)
> +++ llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp Tue Mar 1 04:08:01 2016
> @@ -265,7 +265,8 @@ MipsTargetLowering::MipsTargetLowering(c
> // Without this, every float setcc comes with a AND/OR with the result,
> // we don't want this, since the fpcmp result goes to a flag register,
> // which is used implicitly by brcond and select operations.
> - AddPromotedToType(ISD::SETCC, MVT::i1, MVT::i32);
> + AddPromotedToType(ISD::SETCC, MVT::i1,
> + ABI.AreGprs64bit() ? MVT::i64 : MVT::i32);
>
> // Mips Custom Operations
> setOperationAction(ISD::BR_JT, MVT::Other, Custom);
> @@ -462,9 +463,10 @@ MipsTargetLowering::createFastISel(Funct
>
> EVT MipsTargetLowering::getSetCCResultType(const DataLayout &, LLVMContext &,
> EVT VT) const {
> - if (!VT.isVector())
> - return MVT::i32;
> - return VT.changeVectorElementTypeToInteger();
> + if (VT.isVector())
> + return VT.changeVectorElementTypeToInteger();
> +
> + return ABI.AreGprs64bit() ? MVT::i64 : MVT::i32;
> }
>
> static SDValue performDivRemCombine(SDNode *N, SelectionDAG &DAG,
> @@ -627,19 +629,19 @@ static SDValue performSELECTCombine(SDNo
> if (!TrueC || !True.getValueType().isInteger())
> return SDValue();
>
> - // We'll also ignore MVT::i64 operands as this optimizations proves
> - // to be ineffective because of the required sign extensions as the result
> - // of a SETCC operator is always MVT::i32 for non-vector types.
> - if (True.getValueType() == MVT::i64)
> - return SDValue();
> -
> int64_t Diff = TrueC->getSExtValue() - FalseC->getSExtValue();
>
> // 1) (a < x) ? y : y-1
> // slti $reg1, a, x
> // addiu $reg2, $reg1, y-1
> - if (Diff == 1)
> - return DAG.getNode(ISD::ADD, DL, SetCC.getValueType(), SetCC, False);
> + if (Diff == 1) {
> + if (SetCC.getValueType().getSizeInBits() >
> + False.getValueType().getSizeInBits())
> + SetCC = DAG.getNode(ISD::TRUNCATE, DL, False.getValueType(), SetCC);
> +
> + SDValue Ret = DAG.getNode(ISD::ADD, DL, False.getValueType(), SetCC, False);
> + return Ret;
> + }
>
> // 2) (a < x) ? y-1 : y
> // slti $reg1, a, x
> @@ -649,6 +651,11 @@ static SDValue performSELECTCombine(SDNo
> ISD::CondCode CC = cast<CondCodeSDNode>(SetCC.getOperand(2))->get();
> SetCC = DAG.getSetCC(DL, SetCC.getValueType(), SetCC.getOperand(0),
> SetCC.getOperand(1), ISD::getSetCCInverse(CC, true));
> +
> + if (SetCC.getValueType().getSizeInBits() >
> + True.getValueType().getSizeInBits())
> + SetCC = DAG.getNode(ISD::TRUNCATE, DL, True.getValueType(), SetCC);
> +
> return DAG.getNode(ISD::ADD, DL, SetCC.getValueType(), SetCC, True);
> }
>
> @@ -1027,19 +1034,30 @@ MipsTargetLowering::EmitInstrWithCustomI
> return insertDivByZeroTrap(MI, *BB, *Subtarget.getInstrInfo(), true);
> case Mips::SEL_D:
> return emitSEL_D(MI, BB);
> + case Mips::SEL64_S:
> + return emitSEL64_S(MI, BB);
>
> case Mips::PseudoSELECT_I:
> case Mips::PseudoSELECT_I64:
> case Mips::PseudoSELECT_S:
> case Mips::PseudoSELECT_D32:
> case Mips::PseudoSELECT_D64:
> - return emitPseudoSELECT(MI, BB, false, Mips::BNE);
> + // 64-bit version
> + case Mips::PseudoSELECT64_I:
> + case Mips::PseudoSELECT64_I64:
> + case Mips::PseudoSELECT64_S:
> + case Mips::PseudoSELECT64_D32:
> + case Mips::PseudoSELECT64_D64:
> + return emitPseudoSELECT(MI, BB, false,
> + Subtarget.isGP32bit() ? Mips::BNE : Mips::BNE64);
> +
> case Mips::PseudoSELECTFP_F_I:
> case Mips::PseudoSELECTFP_F_I64:
> case Mips::PseudoSELECTFP_F_S:
> case Mips::PseudoSELECTFP_F_D32:
> case Mips::PseudoSELECTFP_F_D64:
> return emitPseudoSELECT(MI, BB, true, Mips::BC1F);
> +
> case Mips::PseudoSELECTFP_T_I:
> case Mips::PseudoSELECTFP_T_I64:
> case Mips::PseudoSELECTFP_T_S:
> @@ -1574,6 +1592,30 @@ MachineBasicBlock *MipsTargetLowering::e
> return BB;
> }
>
> +MachineBasicBlock *
> +MipsTargetLowering::emitSEL64_S(MachineInstr *MI, MachineBasicBlock *BB) const {
> + MachineFunction *MF = BB->getParent();
> + const TargetRegisterInfo *TRI = Subtarget.getRegisterInfo();
> + const TargetInstrInfo *TII = Subtarget.getInstrInfo();
> + MachineRegisterInfo &RegInfo = MF->getRegInfo();
> + DebugLoc DL = MI->getDebugLoc();
> + MachineBasicBlock::iterator II(MI);
> +
> + unsigned Fc64 = MI->getOperand(1).getReg();
> +
> + const auto &FGR32RegClass = TRI->getRegClass(Mips::FGR32RegClassID);
> + unsigned Fc32 = RegInfo.createVirtualRegister(FGR32RegClass);
> +
> + BuildMI(*BB, II, DL, TII->get(Mips::COPY), Fc32)
> + .addReg(Fc64, 0, Mips::sub_lo);
> +
> + // We don't erase the original instruction, we just replace the condition
> + // register with the 32-bit sub-register.
> + MI->getOperand(1).setReg(Fc32);
> +
> + return BB;
> +}
> +
> //===----------------------------------------------------------------------===//
> // Misc Lower Operation implementation
> //===----------------------------------------------------------------------===//
> @@ -1655,8 +1697,10 @@ SDValue MipsTargetLowering::lowerSETCC(S
> "Floating point operand expected.");
>
> SDLoc DL(Op);
> - SDValue True = DAG.getConstant(1, DL, MVT::i32);
> - SDValue False = DAG.getConstant(0, DL, MVT::i32);
> + EVT VT = getSetCCResultType(DAG.getDataLayout(), *DAG.getContext(),
> + Op.getValueType());
> + SDValue True = DAG.getConstant(1, DL, VT);
> + SDValue False = DAG.getConstant(0, DL, VT);
>
> return createCMovFP(DAG, Cond, True, False, DL);
> }
> @@ -3910,9 +3954,9 @@ MipsTargetLowering::emitPseudoSELECT(Mac
> } else {
> // bne rs, $0, sinkMBB
> BuildMI(BB, DL, TII->get(Opc))
> - .addReg(MI->getOperand(1).getReg())
> - .addReg(Mips::ZERO)
> - .addMBB(sinkMBB);
> + .addReg(MI->getOperand(1).getReg())
> + .addReg(ABI.GetZeroReg())
> + .addMBB(sinkMBB);
> }
>
> // copy0MBB:
>
> Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.h?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsISelLowering.h (original)
> +++ llvm/trunk/lib/Target/Mips/MipsISelLowering.h Tue Mar 1 04:08:01 2016
> @@ -444,6 +444,8 @@ namespace llvm {
> bool IsSRA) const;
> SDValue lowerADD(SDValue Op, SelectionDAG &DAG) const;
> SDValue lowerFP_TO_SINT(SDValue Op, SelectionDAG &DAG) const;
> + SDValue lowerATOMIC_CMP_SWAP_WITH_SUCCESS(SDValue Op,
> + SelectionDAG &DAG) const;
>
> /// isEligibleForTailCallOptimization - Check whether the call is eligible
> /// for tail call optimization.
> @@ -577,6 +579,10 @@ namespace llvm {
> MachineBasicBlock *emitAtomicCmpSwapPartword(MachineInstr *MI,
> MachineBasicBlock *BB, unsigned Size) const;
> MachineBasicBlock *emitSEL_D(MachineInstr *MI, MachineBasicBlock *BB) const;
> +
> + MachineBasicBlock *emitSEL64_S(MachineInstr *MI,
> + MachineBasicBlock *BB) const;
> +
> MachineBasicBlock *emitPseudoSELECT(MachineInstr *MI,
> MachineBasicBlock *BB, bool isFPCmp,
> unsigned Opc) const;
>
> Modified: llvm/trunk/lib/Target/Mips/MipsInstrInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsInstrInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsInstrInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/MipsInstrInfo.td Tue Mar 1 04:08:01 2016
> @@ -1035,17 +1035,18 @@ class CBranchZero<string opstr, DAGOpera
> }
>
> // SetCC
> -class SetCC_R<string opstr, PatFrag cond_op, RegisterOperand RO> :
> - InstSE<(outs GPR32Opnd:$rd), (ins RO:$rs, RO:$rt),
> +class SetCC_R<string opstr, PatFrag cond_op, RegisterOperand RD,
> + RegisterOperand RO> :
> + InstSE<(outs RD:$rd), (ins RO:$rs, RO:$rt),
> !strconcat(opstr, "\t$rd, $rs, $rt"),
> - [(set GPR32Opnd:$rd, (cond_op RO:$rs, RO:$rt))],
> + [(set RD:$rd, (cond_op RO:$rs, RO:$rt))],
> II_SLT_SLTU, FrmR, opstr>;
>
> class SetCC_I<string opstr, PatFrag cond_op, Operand Od, PatLeaf imm_type,
> - RegisterOperand RO>:
> - InstSE<(outs GPR32Opnd:$rt), (ins RO:$rs, Od:$imm16),
> + RegisterOperand RD, RegisterOperand RO>:
> + InstSE<(outs RD:$rt), (ins RO:$rs, Od:$imm16),
> !strconcat(opstr, "\t$rt, $rs, $imm16"),
> - [(set GPR32Opnd:$rt, (cond_op RO:$rs, imm_type:$imm16))],
> + [(set RD:$rt, (cond_op RO:$rs, imm_type:$imm16))],
> II_SLTI_SLTIU, FrmI, opstr>;
>
> // Jump
> @@ -1426,10 +1427,10 @@ def ADDiu : MMRel, StdMMR6Rel, ArithLogi
> }
> def ADDi : MMRel, ArithLogicI<"addi", simm16, GPR32Opnd>, ADDI_FM<0x8>,
> ISA_MIPS1_NOT_32R6_64R6;
> -def SLTi : MMRel, SetCC_I<"slti", setlt, simm16, immSExt16, GPR32Opnd>,
> - SLTI_FM<0xa>;
> -def SLTiu : MMRel, SetCC_I<"sltiu", setult, simm16, immSExt16, GPR32Opnd>,
> - SLTI_FM<0xb>;
> +def SLTi : MMRel, SetCC_I<"slti", setlt, simm16, immSExt16, GPR32Opnd,
> + GPR32Opnd>, SLTI_FM<0xa>;
> +def SLTiu : MMRel, SetCC_I<"sltiu", setult, simm16, immSExt16, GPR32Opnd,
> + GPR32Opnd>, SLTI_FM<0xb>;
> let AdditionalPredicates = [NotInMicroMips] in {
> def ANDi : MMRel, StdMMR6Rel,
> ArithLogicI<"andi", uimm16, GPR32Opnd, II_ANDI, immZExt16, and>,
> @@ -1454,8 +1455,8 @@ def MUL : MMRel, ArithLogicR<"mul", GP
> ADD_FM<0x1c, 2>, ISA_MIPS32_NOT_32R6_64R6;
> def ADD : MMRel, StdMMR6Rel, ArithLogicR<"add", GPR32Opnd>, ADD_FM<0, 0x20>;
> def SUB : MMRel, ArithLogicR<"sub", GPR32Opnd>, ADD_FM<0, 0x22>;
> -def SLT : MMRel, SetCC_R<"slt", setlt, GPR32Opnd>, ADD_FM<0, 0x2a>;
> -def SLTu : MMRel, SetCC_R<"sltu", setult, GPR32Opnd>, ADD_FM<0, 0x2b>;
> +def SLT : MMRel, SetCC_R<"slt", setlt, GPR32Opnd, GPR32Opnd>, ADD_FM<0, 0x2a>;
> +def SLTu : MMRel, SetCC_R<"sltu", setult, GPR32Opnd, GPR32Opnd>, ADD_FM<0, 0x2b>;
> let AdditionalPredicates = [NotInMicroMips] in {
> def AND : MMRel, StdMMR6Rel, ArithLogicR<"and", GPR32Opnd, 1, II_AND, and>,
> ADD_FM<0, 0x24>;
> @@ -2241,37 +2242,40 @@ def : MipsPat<(i32 (extloadi16 addr:$src
> def : MipsPat<(store (i32 0), addr:$dst), (SW ZERO, addr:$dst)>;
>
> // brcond patterns
> -multiclass BrcondPats<RegisterClass RC, Instruction BEQOp, Instruction BNEOp,
> - Instruction SLTOp, Instruction SLTuOp, Instruction SLTiOp,
> - Instruction SLTiuOp, Register ZEROReg> {
> -def : MipsPat<(brcond (i32 (setne RC:$lhs, 0)), bb:$dst),
> +multiclass BrcondPats1<RegisterClass RC, Instruction BEQOp, Instruction BNEOp,
> + Register ZEROReg, ValueType VT> {
> +def : MipsPat<(brcond (VT (setne RC:$lhs, 0)), bb:$dst),
> (BNEOp RC:$lhs, ZEROReg, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (seteq RC:$lhs, 0)), bb:$dst),
> +def : MipsPat<(brcond (VT (seteq RC:$lhs, 0)), bb:$dst),
> (BEQOp RC:$lhs, ZEROReg, bb:$dst)>;
> -
> -def : MipsPat<(brcond (i32 (setge RC:$lhs, RC:$rhs)), bb:$dst),
> - (BEQ (SLTOp RC:$lhs, RC:$rhs), ZERO, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setuge RC:$lhs, RC:$rhs)), bb:$dst),
> - (BEQ (SLTuOp RC:$lhs, RC:$rhs), ZERO, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setge RC:$lhs, immSExt16:$rhs)), bb:$dst),
> - (BEQ (SLTiOp RC:$lhs, immSExt16:$rhs), ZERO, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setuge RC:$lhs, immSExt16:$rhs)), bb:$dst),
> - (BEQ (SLTiuOp RC:$lhs, immSExt16:$rhs), ZERO, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setgt RC:$lhs, immSExt16Plus1:$rhs)), bb:$dst),
> - (BEQ (SLTiOp RC:$lhs, (Plus1 imm:$rhs)), ZERO, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setugt RC:$lhs, immSExt16Plus1:$rhs)), bb:$dst),
> - (BEQ (SLTiuOp RC:$lhs, (Plus1 imm:$rhs)), ZERO, bb:$dst)>;
> -
> -def : MipsPat<(brcond (i32 (setle RC:$lhs, RC:$rhs)), bb:$dst),
> - (BEQ (SLTOp RC:$rhs, RC:$lhs), ZERO, bb:$dst)>;
> -def : MipsPat<(brcond (i32 (setule RC:$lhs, RC:$rhs)), bb:$dst),
> - (BEQ (SLTuOp RC:$rhs, RC:$lhs), ZERO, bb:$dst)>;
> -
> def : MipsPat<(brcond RC:$cond, bb:$dst),
> (BNEOp RC:$cond, ZEROReg, bb:$dst)>;
> }
>
> -defm : BrcondPats<GPR32, BEQ, BNE, SLT, SLTu, SLTi, SLTiu, ZERO>;
> +multiclass BrcondPats2<RegisterClass RC, Instruction BEQOp, Register ZEROReg,
> + Instruction SLTOp, Instruction SLTuOp,
> + Instruction SLTiOp, Instruction SLTiuOp,
> + ValueType VT> {
> +def : MipsPat<(brcond (VT (setge RC:$lhs, RC:$rhs)), bb:$dst),
> + (BEQOp (SLTOp RC:$lhs, RC:$rhs), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setuge RC:$lhs, RC:$rhs)), bb:$dst),
> + (BEQOp (SLTuOp RC:$lhs, RC:$rhs), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setge RC:$lhs, immSExt16:$rhs)), bb:$dst),
> + (BEQOp (SLTiOp RC:$lhs, immSExt16:$rhs), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setuge RC:$lhs, immSExt16:$rhs)), bb:$dst),
> + (BEQOp (SLTiuOp RC:$lhs, immSExt16:$rhs), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setgt RC:$lhs, immSExt16Plus1:$rhs)), bb:$dst),
> + (BEQOp (SLTiOp RC:$lhs, (Plus1 imm:$rhs)), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setugt RC:$lhs, immSExt16Plus1:$rhs)), bb:$dst),
> + (BEQOp (SLTiuOp RC:$lhs, (Plus1 imm:$rhs)), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setle RC:$lhs, RC:$rhs)), bb:$dst),
> + (BEQOp (SLTOp RC:$rhs, RC:$lhs), ZEROReg, bb:$dst)>;
> +def : MipsPat<(brcond (VT (setule RC:$lhs, RC:$rhs)), bb:$dst),
> + (BEQOp (SLTuOp RC:$rhs, RC:$lhs), ZEROReg, bb:$dst)>;
> +}
> +
> +defm : BrcondPats1<GPR32, BEQ, BNE, ZERO, i32>;
> +defm : BrcondPats2<GPR32, BEQ, ZERO, SLT, SLTu, SLTi, SLTiu, i32>;
>
> def : MipsPat<(brcond (i32 (setlt i32:$lhs, 1)), bb:$dst),
> (BLEZ i32:$lhs, bb:$dst)>;
> @@ -2291,11 +2295,12 @@ multiclass SeteqPats<RegisterClass RC, I
> (SLTuOp ZEROReg, (XOROp RC:$lhs, RC:$rhs))>;
> }
>
> -multiclass SetlePats<RegisterClass RC, Instruction SLTOp, Instruction SLTuOp> {
> +multiclass SetlePats<RegisterClass RC, Instruction SLTOp, Instruction SLTuOp,
> + Instruction XORiOp> {
> def : MipsPat<(setle RC:$lhs, RC:$rhs),
> - (XORi (SLTOp RC:$rhs, RC:$lhs), 1)>;
> + (XORiOp (SLTOp RC:$rhs, RC:$lhs), 1)>;
> def : MipsPat<(setule RC:$lhs, RC:$rhs),
> - (XORi (SLTuOp RC:$rhs, RC:$lhs), 1)>;
> + (XORiOp (SLTuOp RC:$rhs, RC:$lhs), 1)>;
> }
>
> multiclass SetgtPats<RegisterClass RC, Instruction SLTOp, Instruction SLTuOp> {
> @@ -2305,26 +2310,27 @@ multiclass SetgtPats<RegisterClass RC, I
> (SLTuOp RC:$rhs, RC:$lhs)>;
> }
>
> -multiclass SetgePats<RegisterClass RC, Instruction SLTOp, Instruction SLTuOp> {
> +multiclass SetgePats<RegisterClass RC, Instruction SLTOp, Instruction SLTuOp,
> + Instruction XORiOp> {
> def : MipsPat<(setge RC:$lhs, RC:$rhs),
> - (XORi (SLTOp RC:$lhs, RC:$rhs), 1)>;
> + (XORiOp (SLTOp RC:$lhs, RC:$rhs), 1)>;
> def : MipsPat<(setuge RC:$lhs, RC:$rhs),
> - (XORi (SLTuOp RC:$lhs, RC:$rhs), 1)>;
> + (XORiOp (SLTuOp RC:$lhs, RC:$rhs), 1)>;
> }
>
> multiclass SetgeImmPats<RegisterClass RC, Instruction SLTiOp,
> - Instruction SLTiuOp> {
> + Instruction SLTiuOp, Instruction XORiOp> {
> def : MipsPat<(setge RC:$lhs, immSExt16:$rhs),
> - (XORi (SLTiOp RC:$lhs, immSExt16:$rhs), 1)>;
> + (XORiOp (SLTiOp RC:$lhs, immSExt16:$rhs), 1)>;
> def : MipsPat<(setuge RC:$lhs, immSExt16:$rhs),
> - (XORi (SLTiuOp RC:$lhs, immSExt16:$rhs), 1)>;
> + (XORiOp (SLTiuOp RC:$lhs, immSExt16:$rhs), 1)>;
> }
>
> defm : SeteqPats<GPR32, SLTiu, XOR, SLTu, ZERO>;
> -defm : SetlePats<GPR32, SLT, SLTu>;
> +defm : SetlePats<GPR32, SLT, SLTu, XORi>;
> defm : SetgtPats<GPR32, SLT, SLTu>;
> -defm : SetgePats<GPR32, SLT, SLTu>;
> -defm : SetgeImmPats<GPR32, SLTi, SLTiu>;
> +defm : SetgePats<GPR32, SLT, SLTu, XORi>;
> +defm : SetgeImmPats<GPR32, SLTi, SLTiu, XORi>;
>
> // bswap pattern
> def : MipsPat<(bswap GPR32:$rt), (ROTR (WSBH GPR32:$rt), 16)>;
>
> Modified: llvm/trunk/lib/Target/Mips/MipsRegisterInfo.td
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsRegisterInfo.td?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsRegisterInfo.td (original)
> +++ llvm/trunk/lib/Target/Mips/MipsRegisterInfo.td Tue Mar 1 04:08:01 2016
> @@ -391,8 +391,9 @@ def FCC : RegisterClass<"Mips", [i32], 3
> Unallocatable;
>
> // MIPS32r6/MIPS64r6 store FPU condition codes in normal FGR registers.
> -// This class allows us to represent this in codegen patterns.
> -def FGRCC : RegisterClass<"Mips", [i32], 32, (sequence "F%u", 0, 31)>;
> +// These classes allow us to represent this in codegen patterns.
> +def FGRCC32 : RegisterClass<"Mips", [i32], 32, (sequence "F%u", 0, 31)>;
> +def FGRCC64 : RegisterClass<"Mips", [i64], 64, (sequence "D%u_64", 0, 31)>;
>
> def MSA128B: RegisterClass<"Mips", [v16i8], 128,
> (sequence "W%u", 0, 31)>;
> @@ -595,7 +596,13 @@ def FGR32Opnd : RegisterOperand<FGR32> {
> let ParserMatchClass = FGR32AsmOperand;
> }
>
> -def FGRCCOpnd : RegisterOperand<FGRCC> {
> +def FGRCC32Opnd : RegisterOperand<FGRCC32> {
> + // The assembler doesn't use register classes so we can re-use
> + // FGR32AsmOperand.
> + let ParserMatchClass = FGR32AsmOperand;
> +}
> +
> +def FGRCC64Opnd : RegisterOperand<FGRCC64> {
> // The assembler doesn't use register classes so we can re-use
> // FGR32AsmOperand.
> let ParserMatchClass = FGR32AsmOperand;
>
> Modified: llvm/trunk/lib/Target/Mips/MipsSEISelDAGToDAG.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsSEISelDAGToDAG.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsSEISelDAGToDAG.cpp (original)
> +++ llvm/trunk/lib/Target/Mips/MipsSEISelDAGToDAG.cpp Tue Mar 1 04:08:01 2016
> @@ -251,28 +251,14 @@ SDNode *MipsSEDAGToDAGISel::selectAddESu
> SDValue LHS = Node->getOperand(0), RHS = Node->getOperand(1);
> EVT VT = LHS.getValueType();
>
> - SDNode *Carry = CurDAG->getMachineNode(SLTuOp, DL, VT, Ops);
> -
> - if (Subtarget->isGP64bit()) {
> - // On 64-bit targets, sltu produces an i64 but our backend currently says
> - // that SLTu64 produces an i32. We need to fix this in the long run but for
> - // now, just make the DAG type-correct by asserting the upper bits are zero.
> - Carry = CurDAG->getMachineNode(Mips::SUBREG_TO_REG, DL, VT,
> - CurDAG->getTargetConstant(0, DL, VT),
> - SDValue(Carry, 0),
> - CurDAG->getTargetConstant(Mips::sub_32, DL,
> - VT));
> - }
> -
> // Generate a second addition only if we know that RHS is not a
> // constant-zero node.
> - SDNode *AddCarry = Carry;
> + SDNode *Carry = CurDAG->getMachineNode(SLTuOp, DL, VT, Ops);
> ConstantSDNode *C = dyn_cast<ConstantSDNode>(RHS);
> if (!C || C->getZExtValue())
> - AddCarry = CurDAG->getMachineNode(ADDuOp, DL, VT, SDValue(Carry, 0), RHS);
> + Carry = CurDAG->getMachineNode(ADDuOp, DL, VT, SDValue(Carry, 0), RHS);
>
> - return CurDAG->SelectNodeTo(Node, MOp, VT, MVT::Glue, LHS,
> - SDValue(AddCarry, 0));
> + return CurDAG->SelectNodeTo(Node, MOp, VT, MVT::Glue, LHS, SDValue(Carry, 0));
> }
>
> /// Match frameindex
>
> Modified: llvm/trunk/lib/Target/Mips/MipsSEISelLowering.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsSEISelLowering.cpp?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/Mips/MipsSEISelLowering.cpp (original)
> +++ llvm/trunk/lib/Target/Mips/MipsSEISelLowering.cpp Tue Mar 1 04:08:01 2016
> @@ -1292,7 +1292,6 @@ SDValue MipsSETargetLowering::lowerMulDi
> return DAG.getMergeValues(Vals, DL);
> }
>
> -
> static SDValue initAccumulator(SDValue In, SDLoc DL, SelectionDAG &DAG) {
> SDValue InLo = DAG.getNode(ISD::EXTRACT_ELEMENT, DL, MVT::i32, In,
> DAG.getConstant(0, DL, MVT::i32));
>
> Modified: llvm/trunk/test/CodeGen/Mips/atomic.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/atomic.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/atomic.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/atomic.ll Tue Mar 1 04:08:01 2016
> @@ -344,7 +344,11 @@ entry:
> ; HAS-SEB-SEH: seb $[[R19:[0-9]+]], $[[R17]]
>
> ; ALL: xor $[[R20:[0-9]+]], $[[R19]], $5
> -; ALL: sltiu $2, $[[R20]], 1
> +
> +; MIPS32-ANY: sltiu $2, $[[R20]], 1
> +
> +; MIPS64-ANY: sltiu $[[R21:[0-9]+]], $[[R20]], 1
> +; MIPS64-ANY: sll $2, $[[R21]], 0
> }
>
> ; Check one i16 so that we cover the seh sign extend
>
> Modified: llvm/trunk/test/CodeGen/Mips/blez_bgez.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/blez_bgez.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/blez_bgez.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/blez_bgez.ll Tue Mar 1 04:08:01 2016
> @@ -1,10 +1,13 @@
> -; RUN: llc -march=mipsel < %s | FileCheck %s
> -; RUN: llc -march=mips64el < %s | FileCheck %s
> +; RUN: llc -march=mipsel < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL
> +; RUN: llc -march=mips64el < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=GP64
>
> -; CHECK-LABEL: test_blez:
> -; CHECK: blez ${{[0-9]+}}, $BB
> +declare void @foo1()
>
> define void @test_blez(i32 %a) {
> +; ALL-LABEL: test_blez:
> +; ALL: blez ${{[0-9]+}}, $BB
> entry:
> %cmp = icmp sgt i32 %a, 0
> br i1 %cmp, label %if.then, label %if.end
> @@ -17,17 +20,44 @@ if.end:
> ret void
> }
>
> -declare void @foo1()
> -
> -; CHECK-LABEL: test_bgez:
> -; CHECK: bgez ${{[0-9]+}}, $BB
> -
> define void @test_bgez(i32 %a) {
> entry:
> +; ALL-LABEL: test_bgez:
> +; ALL: bgez ${{[0-9]+}}, $BB
> %cmp = icmp slt i32 %a, 0
> br i1 %cmp, label %if.then, label %if.end
>
> if.then:
> + tail call void @foo1()
> + br label %if.end
> +
> +if.end:
> + ret void
> +}
> +
> +define void @test_blez_64(i64 %a) {
> +; GP64-LABEL: test_blez_64:
> +; GP64: blez ${{[0-9]+}}, $BB
> +entry:
> + %cmp = icmp sgt i64 %a, 0
> + br i1 %cmp, label %if.then, label %if.end
> +
> +if.then:
> + tail call void @foo1()
> + br label %if.end
> +
> +if.end:
> + ret void
> +}
> +
> +define void @test_bgez_64(i64 %a) {
> +entry:
> +; ALL-LABEL: test_bgez_64:
> +; ALL: bgez ${{[0-9]+}}, $BB
> + %cmp = icmp slt i64 %a, 0
> + br i1 %cmp, label %if.then, label %if.end
> +
> +if.then:
> tail call void @foo1()
> br label %if.end
>
>
> Modified: llvm/trunk/test/CodeGen/Mips/cmov.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/cmov.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/cmov.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/cmov.ll Tue Mar 1 04:08:01 2016
> @@ -1,10 +1,17 @@
> -; RUN: llc -march=mips -mcpu=mips32 < %s | FileCheck %s -check-prefix=ALL -check-prefix=32-CMOV
> -; RUN: llc -march=mips -mcpu=mips32 -regalloc=basic < %s | FileCheck %s -check-prefix=ALL -check-prefix=32-CMOV
> -; RUN: llc -march=mips -mcpu=mips32r2 < %s | FileCheck %s -check-prefix=ALL -check-prefix=32-CMOV
> -; RUN: llc -march=mips -mcpu=mips32r6 < %s | FileCheck %s -check-prefix=ALL -check-prefix=32-CMP
> -; RUN: llc -march=mips64el -mcpu=mips4 < %s | FileCheck %s -check-prefix=ALL -check-prefix=64-CMOV
> -; RUN: llc -march=mips64el -mcpu=mips64 < %s | FileCheck %s -check-prefix=ALL -check-prefix=64-CMOV
> -; RUN: llc -march=mips64el -mcpu=mips64r6 < %s | FileCheck %s -check-prefix=ALL -check-prefix=64-CMP
> +; RUN: llc -march=mips -mcpu=mips32 < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=32-CMOV
> +; RUN: llc -march=mips -mcpu=mips32 -regalloc=basic < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=32-CMOV
> +; RUN: llc -march=mips -mcpu=mips32r2 < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=32-CMOV
> +; RUN: llc -march=mips -mcpu=mips32r6 < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=32-CMP
> +; RUN: llc -march=mips64el -mcpu=mips4 < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=64-CMOV -check-prefix=64
> +; RUN: llc -march=mips64el -mcpu=mips64 < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=64-CMOV -check-prefix=64
> +; RUN: llc -march=mips64el -mcpu=mips64r6 < %s | \
> +; RUN: FileCheck %s -check-prefix=ALL -check-prefix=64-CMP -check-prefix=64
>
> @i1 = global [3 x i32] [i32 1, i32 2, i32 3], align 4
> @i3 = common global i32* null, align 4
> @@ -413,20 +420,9 @@ entry:
> ; 32-CMP-DAG: or $3, $[[T1]], $[[T0]]
> ; 32-CMP-DAG: addiu $2, $zero, 0
>
> -; 64-CMOV-DAG: addiu $[[I5:[0-9]+]], $zero, 5
> -; 64-CMOV-DAG: addiu $[[I4:2]], $zero, 4
> -; 64-CMOV-DAG: slti $[[R0:[0-9]+]], $4, 32767
> -; 64-CMOV-DAG: movz $[[I4]], $[[I5]], $[[R0]]
> -
> -; 64-CMP-DAG: addiu $[[I5:[0-9]+]], $zero, 5
> -; 64-CMP-DAG: addiu $[[I4:[0-9]+]], $zero, 4
> -; 64-CMP-DAG: slti $[[R0:[0-9]+]], $4, 32767
> -; FIXME: We can do better than this by adding/subtracting the result of slti
> -; to/from one of the constants.
> -; 64-CMP-DAG: seleqz $[[T0:[0-9]+]], $[[I5]], $[[R0]]
> -; 64-CMP-DAG: selnez $[[T1:[0-9]+]], $[[I4]], $[[R0]]
> -; 64-CMP-DAG: or $2, $[[T0]], $[[T1]]
> -
> +; 64-DAG: daddiu $[[T0:[0-9]+]], $zero, 32766
> +; 64-DAG: slt $[[T1:[0-9]+]], $[[T0]], $4
> +; 64-DAG: ori $2, $[[T1]], 4
> define i64 @slti64_0(i64 %a) {
> entry:
> %cmp = icmp sgt i64 %a, 32766
> @@ -458,21 +454,9 @@ entry:
> ; 32-CMP-DAG: or $3, $[[T1]], $[[T0]]
> ; 32-CMP-DAG: addiu $2, $zero, 0
>
> -; 64-CMOV-DAG: daddiu $[[I5:[0-9]+]], $zero, 5
> -; 64-CMOV-DAG: daddiu $[[I4:2]], $zero, 4
> -; 64-CMOV-DAG: daddiu $[[R1:[0-9]+]], $zero, 32767
> -; 64-CMOV-DAG: slt $[[R0:[0-9]+]], $[[R1]], $4
> -; 64-CMOV-DAG: movn $[[I4]], $[[I5]], $[[R0]]
> -
> -; 64-CMP-DAG: daddiu $[[I5:[0-9]+]], $zero, 5
> -; 64-CMP-DAG: daddiu $[[I4:2]], $zero, 4
> -; 64-CMP-DAG: daddiu $[[R1:[0-9]+]], $zero, 32767
> -; 64-CMP-DAG: slt $[[R0:[0-9]+]], $[[R1]], $4
> -; FIXME: We can do better than this by using selccz to choose between -0 and -2
> -; 64-CMP-DAG: selnez $[[T0:[0-9]+]], $[[I5]], $[[R0]]
> -; 64-CMP-DAG: seleqz $[[T1:[0-9]+]], $[[I4]], $[[R0]]
> -; 64-CMP-DAG: or $2, $[[T0]], $[[T1]]
> -
> +; 64-DAG: daddiu $[[T0:[0-9]+]], $zero, 32767
> +; 64-DAG: slt $[[T1:[0-9]+]], $[[T0]], $4
> +; 64-DAG: ori $2, $[[T1]], 4
> define i64 @slti64_1(i64 %a) {
> entry:
> %cmp = icmp sgt i64 %a, 32767
> @@ -480,27 +464,23 @@ entry:
> ret i64 %conv
> }
>
> -; ALL-LABEL: slti64_2:
> +; ALL-LABEL: slti32_2:
>
> -; FIXME: The 32-bit versions of this test are too complicated to reasonably
> -; match at the moment. They do show some missing optimizations though
> -; such as:
> -; (movz $a, $b, (neg $c)) -> (movn $a, $b, $c)
> -
> -; 64-CMOV-DAG: addiu $[[I3:[0-9]+]], $zero, 3
> -; 64-CMOV-DAG: addiu $[[I4:2]], $zero, 4
> -; 64-CMOV-DAG: slti $[[R0:[0-9]+]], $4, -32768
> -; 64-CMOV-DAG: movz $[[I4]], $[[I3]], $[[R0]]
> +; FIXME: Remove unecessary sign-extension.
> +; 64-DAG: slti $[[T0:[0-9]+]], $4, -32768
> +; 64-DAG: sll $[[T1:[0-9]+]], $[[T0]], 0
> +; 64-DAG: addiu $2, $[[T1]], 3
> +define i32 @slti32_2(i32 signext %a) {
> +entry:
> + %cmp = icmp sgt i32 %a, -32769
> + %conv = select i1 %cmp, i32 3, i32 4
> + ret i32 %conv
> +}
>
> -; 64-CMP-DAG: addiu $[[I3:[0-9]+]], $zero, 3
> -; 64-CMP-DAG: addiu $[[I4:[0-9]+]], $zero, 4
> -; 64-CMP-DAG: slti $[[R0:[0-9]+]], $4, -32768
> -; FIXME: We can do better than this by adding/subtracting the result of slti
> -; to/from one of the constants.
> -; 64-CMP-DAG: seleqz $[[T0:[0-9]+]], $[[I3]], $[[R0]]
> -; 64-CMP-DAG: selnez $[[T1:[0-9]+]], $[[I4]], $[[R0]]
> -; 64-CMP-DAG: or $2, $[[T0]], $[[T1]]
> +; ALL-LABEL: slti64_2:
>
> +; 64-DAG: slti $[[T0:[0-9]+]], $4, -32768
> +; 64-DAG: daddiu $2, $[[T0]], 3
> define i64 @slti64_2(i64 %a) {
> entry:
> %cmp = icmp sgt i64 %a, -32769
> @@ -508,28 +488,24 @@ entry:
> ret i64 %conv
> }
>
> -; ALL-LABEL: slti64_3:
> +; ALL-LABEL: slti32_3:
>
> -; FIXME: The 32-bit versions of this test are too complicated to reasonably
> -; match at the moment. They do show some missing optimizations though
> -; such as:
> -; (movz $a, $b, (neg $c)) -> (movn $a, $b, $c)
> -
> -; 64-CMOV-DAG: daddiu $[[I5:[0-9]+]], $zero, 5
> -; 64-CMOV-DAG: daddiu $[[I4:2]], $zero, 4
> -; 64-CMOV-DAG: daddiu $[[R1:[0-9]+]], ${{[0-9]+}}, 32766
> -; 64-CMOV-DAG: slt $[[R0:[0-9]+]], $[[R1]], $4
> -; 64-CMOV-DAG: movn $[[I4]], $[[I5]], $[[R0]]
> +; FIXME: Remove unecessary sign-extension.
> +; 64-DAG: slt $[[R0:[0-9]+]], ${{[0-9]+}}, $4
> +; 64-DAG: sll $[[R1:[0-9]+]], $[[R0]], 0
> +; 64-DAG: ori $2, $[[R1]], 4
> +define i32 @slti32_3(i32 signext %a) {
> +entry:
> + %cmp = icmp sgt i32 %a, -32770
> + %conv = select i1 %cmp, i32 5, i32 4
> + ret i32 %conv
> +}
>
> -; 64-CMP-DAG: daddiu $[[I5:[0-9]+]], $zero, 5
> -; 64-CMP-DAG: daddiu $[[I4:2]], $zero, 4
> -; 64-CMP-DAG: daddiu $[[R1:[0-9]+]], ${{[0-9]+}}, 32766
> -; 64-CMP-DAG: slt $[[R0:[0-9]+]], $[[R1]], $4
> -; FIXME: We can do better than this by using selccz to choose between -0 and -2
> -; 64-CMP-DAG: selnez $[[T0:[0-9]+]], $[[I5]], $[[R0]]
> -; 64-CMP-DAG: seleqz $[[T1:[0-9]+]], $[[I4]], $[[R0]]
> -; 64-CMP-DAG: or $2, $[[T0]], $[[T1]]
> +; ALL-LABEL: slti64_3:
>
> +; 64-DAG: daddiu $[[R0:[0-9]+]], ${{[0-9]+}}, 32766
> +; 64-DAG: slt $[[R1:[0-9]+]], $[[R0]], $4
> +; 64-DAG: ori $2, $[[R1]], 4
> define i64 @slti64_3(i64 %a) {
> entry:
> %cmp = icmp sgt i64 %a, -32770
>
> Modified: llvm/trunk/test/CodeGen/Mips/countleading.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/countleading.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/countleading.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/countleading.ll Tue Mar 1 04:08:01 2016
> @@ -4,7 +4,7 @@
> ; RUN: llc -march=mips64el -mcpu=mips4 < %s | FileCheck -check-prefix=ALL -check-prefix=MIPS4 %s
> ; RUN: llc -march=mips64el -mcpu=mips64 < %s | FileCheck -check-prefix=ALL -check-prefix=MIPS64-GT-R1 %s
> ; RUN: llc -march=mips64el -mcpu=mips64r2 < %s | FileCheck -check-prefix=ALL -check-prefix=MIPS64-GT-R1 %s
> -; R!N: llc -march=mips64el -mcpu=mips64r6 < %s | FileCheck -check-prefix=ALL -check-prefix=MIPS64-GT-R1 %s
> +; RUN: llc -march=mips64el -mcpu=mips64r6 < %s | FileCheck -check-prefix=ALL -check-prefix=MIPS64-GT-R1 %s
>
> ; Prefixes:
> ; ALL - All
>
> Modified: llvm/trunk/test/CodeGen/Mips/fcmp.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/fcmp.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/fcmp.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/fcmp.ll Tue Mar 1 04:08:01 2016
> @@ -29,17 +29,21 @@ define i32 @oeq_f32(float %a, float %b)
> ; 32-C-DAG: c.eq.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.eq.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.eq.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> ; 32-CMP-DAG: andi $2, $[[T1]], 1
>
> +; FIXME: The sign extension below is redundant.
> ; 64-CMP-DAG: cmp.eq.s $[[T0:f[0-9]+]], $f12, $f13
> -; 64-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> -; 64-CMP-DAG: andi $2, $[[T1]], 1
> +; 64-CMP-DAG: dmfc1 $[[T1:[0-9]+]], $[[T0]]
> +; 64-CMP-DAG: sll $[[T2:[0-9]+]], $[[T1]], 0
> +; 64-CMP-DAG: andi $2, $[[T2]], 1
>
> %1 = fcmp oeq float %a, %b
> %2 = zext i1 %1 to i32
> @@ -53,9 +57,11 @@ define i32 @ogt_f32(float %a, float %b)
> ; 32-C-DAG: c.ule.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ule.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.lt.s $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -77,9 +83,11 @@ define i32 @oge_f32(float %a, float %b)
> ; 32-C-DAG: c.ult.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ult.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.le.s $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -101,9 +109,11 @@ define i32 @olt_f32(float %a, float %b)
> ; 32-C-DAG: c.olt.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.olt.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.lt.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -125,9 +135,11 @@ define i32 @ole_f32(float %a, float %b)
> ; 32-C-DAG: c.ole.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ole.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.le.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -149,9 +161,11 @@ define i32 @one_f32(float %a, float %b)
> ; 32-C-DAG: c.ueq.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ueq.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ueq.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -175,9 +189,11 @@ define i32 @ord_f32(float %a, float %b)
> ; 32-C-DAG: c.un.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.un.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.un.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -201,9 +217,11 @@ define i32 @ueq_f32(float %a, float %b)
> ; 32-C-DAG: c.ueq.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ueq.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ueq.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -225,9 +243,11 @@ define i32 @ugt_f32(float %a, float %b)
> ; 32-C-DAG: c.ole.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ole.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ult.s $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -249,9 +269,11 @@ define i32 @uge_f32(float %a, float %b)
> ; 32-C-DAG: c.olt.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.olt.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ule.s $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -273,9 +295,11 @@ define i32 @ult_f32(float %a, float %b)
> ; 32-C-DAG: c.ult.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ult.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ult.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -298,9 +322,11 @@ define i32 @ule_f32(float %a, float %b)
> ; 32-C-DAG: c.ule.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ule.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ule.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -322,9 +348,11 @@ define i32 @une_f32(float %a, float %b)
> ; 32-C-DAG: c.eq.s $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.eq.s $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.eq.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -348,9 +376,11 @@ define i32 @uno_f32(float %a, float %b)
> ; 32-C-DAG: c.un.s $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.un.s $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.un.s $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -390,9 +420,11 @@ define i32 @oeq_f64(double %a, double %b
> ; 32-C-DAG: c.eq.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.eq.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.eq.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -414,9 +446,11 @@ define i32 @ogt_f64(double %a, double %b
> ; 32-C-DAG: c.ule.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ule.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.lt.d $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -438,9 +472,11 @@ define i32 @oge_f64(double %a, double %b
> ; 32-C-DAG: c.ult.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ult.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.le.d $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -462,9 +498,11 @@ define i32 @olt_f64(double %a, double %b
> ; 32-C-DAG: c.olt.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.olt.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.lt.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -486,9 +524,11 @@ define i32 @ole_f64(double %a, double %b
> ; 32-C-DAG: c.ole.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ole.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.le.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -510,9 +550,11 @@ define i32 @one_f64(double %a, double %b
> ; 32-C-DAG: c.ueq.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ueq.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ueq.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -536,9 +578,11 @@ define i32 @ord_f64(double %a, double %b
> ; 32-C-DAG: c.un.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.un.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.un.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -562,9 +606,11 @@ define i32 @ueq_f64(double %a, double %b
> ; 32-C-DAG: c.ueq.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ueq.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ueq.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -586,9 +632,11 @@ define i32 @ugt_f64(double %a, double %b
> ; 32-C-DAG: c.ole.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ole.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ult.d $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -610,9 +658,11 @@ define i32 @uge_f64(double %a, double %b
> ; 32-C-DAG: c.olt.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.olt.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ule.d $[[T0:f[0-9]+]], $f14, $f12
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -634,9 +684,11 @@ define i32 @ult_f64(double %a, double %b
> ; 32-C-DAG: c.ult.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ult.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ult.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -658,9 +710,11 @@ define i32 @ule_f64(double %a, double %b
> ; 32-C-DAG: c.ule.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.ule.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.ule.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -682,9 +736,11 @@ define i32 @une_f64(double %a, double %b
> ; 32-C-DAG: c.eq.d $f12, $f14
> ; 32-C: movt $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.eq.d $f12, $f13
> -; 64-C: movt $2, $zero, $fcc0
> +; 64-C-DAG: movt $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.eq.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
> @@ -708,9 +764,11 @@ define i32 @uno_f64(double %a, double %b
> ; 32-C-DAG: c.un.d $f12, $f14
> ; 32-C: movf $2, $zero, $fcc0
>
> -; 64-C-DAG: addiu $2, $zero, 1
> +; FIXME: Remove redundant sign extension.
> +; 64-C-DAG: daddiu $[[T0:[0-9]+]], $zero, 1
> ; 64-C-DAG: c.un.d $f12, $f13
> -; 64-C: movf $2, $zero, $fcc0
> +; 64-C-DAG: movf $[[T0]], $zero, $fcc0
> +; 64-C: sll $2, $[[T0]], 0
>
> ; 32-CMP-DAG: cmp.un.d $[[T0:f[0-9]+]], $f12, $f14
> ; 32-CMP-DAG: mfc1 $[[T1:[0-9]+]], $[[T0]]
>
> Modified: llvm/trunk/test/CodeGen/Mips/llvm-ir/select-dbl.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/llvm-ir/select-dbl.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/llvm-ir/select-dbl.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/llvm-ir/select-dbl.ll Tue Mar 1 04:08:01 2016
> @@ -29,7 +29,7 @@
> ; RUN: llc < %s -march=mips64 -mcpu=mips64r6 | FileCheck %s \
> ; RUN: -check-prefix=ALL -check-prefix=SEL -check-prefix=SEL-64
>
> -define double @tst_select_i1_double(i1 signext %s, double %x, double %y) {
> +define double @tst_select_i1_double(i1 %s, double %x, double %y) {
> entry:
> ; ALL-LABEL: tst_select_i1_double:
>
> @@ -53,8 +53,10 @@ entry:
>
> ; SEL-32: mtc1 $7, $[[F0:f[0-9]+]]
> ; SEL-32: mthc1 $6, $[[F0]]
> + ; SEL-32: sll $[[T0:[0-9]+]], $4, 31
> + ; SEL-32: sra $[[T1:[0-9]+]], $[[T0]], 31
> ; SEL-32: ldc1 $[[F1:f[0-9]+]], 16($sp)
> - ; SEL-32: mtc1 $4, $f0
> + ; SEL-32: mtc1 $[[T1]], $f0
> ; SEL-32: sel.d $f0, $[[F1]], $[[F0]]
>
> ; M3: andi $[[T0:[0-9]+]], $4, 1
> @@ -69,14 +71,15 @@ entry:
> ; CMOV-64: movn.d $f14, $f13, $[[T0]]
> ; CMOV-64: mov.d $f0, $f14
>
> - ; SEL-64: mtc1 $4, $f0
> + ; SEL-64: dsll $[[T0:[0-9]+]], $4, 63
> + ; SEL-64: dsra $[[T1:[0-9]+]], $[[T0]], 63
> + ; SEL-64: dmtc1 $[[T1]], $f0
> ; SEL-64: sel.d $f0, $f14, $f13
> %r = select i1 %s, double %x, double %y
> ret double %r
> }
>
> -define double @tst_select_i1_double_reordered(double %x, double %y,
> - i1 signext %s) {
> +define double @tst_select_i1_double_reordered(double %x, double %y, i1 %s) {
> entry:
> ; ALL-LABEL: tst_select_i1_double_reordered:
>
> @@ -110,7 +113,9 @@ entry:
> ; CMOV-64: movn.d $f13, $f12, $[[T0]]
> ; CMOV-64: mov.d $f0, $f13
>
> - ; SEL-64: mtc1 $6, $f0
> + ; SEL-64: dsll $[[T0:[0-9]+]], $6, 63
> + ; SEL-64: dsra $[[T1:[0-9]+]], $[[T0]], 63
> + ; SEL-64: dmtc1 $[[T1]], $f0
> ; SEL-64: sel.d $f0, $f13, $f12
> %r = select i1 %s, double %x, double %y
> ret double %r
>
> Modified: llvm/trunk/test/CodeGen/Mips/llvm-ir/select-flt.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/llvm-ir/select-flt.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/llvm-ir/select-flt.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/llvm-ir/select-flt.ll Tue Mar 1 04:08:01 2016
> @@ -29,7 +29,7 @@
> ; RUN: llc < %s -march=mips64 -mcpu=mips64r6 | FileCheck %s \
> ; RUN: -check-prefix=ALL -check-prefix=SEL -check-prefix=SEL-64
>
> -define float @tst_select_i1_float(i1 signext %s, float %x, float %y) {
> +define float @tst_select_i1_float(i1 %s, float %x, float %y) {
> entry:
> ; ALL-LABEL: tst_select_i1_float:
>
> @@ -51,21 +51,24 @@ entry:
>
> ; SEL-32: mtc1 $5, $[[F0:f[0-9]+]]
> ; SEL-32: mtc1 $6, $[[F1:f[0-9]+]]
> - ; SEL-32: mtc1 $4, $f0
> + ; SEL-32: sll $[[T0:[0-9]+]], $4, 31
> + ; SEL-32: sra $[[T1:[0-9]+]], $[[T0]], 31
> + ; SEL-32: mtc1 $[[T1]], $f0
> ; SEL-32: sel.s $f0, $[[F1]], $[[F0]]
>
> ; CMOV-64: andi $[[T0:[0-9]+]], $4, 1
> ; CMOV-64: movn.s $f14, $f13, $[[T0]]
> ; CMOV-64: mov.s $f0, $f14
>
> - ; SEL-64: mtc1 $4, $f0
> + ; SEL-64: dsll $[[T0:[0-9]+]], $4, 63
> + ; SEL-64: dsra $[[T1:[0-9]+]], $[[T0]], 63
> + ; SEL-64: dmtc1 $[[T1]], $f0
> ; SEL-64: sel.s $f0, $f14, $f13
> %r = select i1 %s, float %x, float %y
> ret float %r
> }
>
> -define float @tst_select_i1_float_reordered(float %x, float %y,
> - i1 signext %s) {
> +define float @tst_select_i1_float_reordered(float %x, float %y, i1 %s) {
> entry:
> ; ALL-LABEL: tst_select_i1_float_reordered:
>
> @@ -82,14 +85,18 @@ entry:
> ; CMOV-32: movn.s $f14, $f12, $[[T0]]
> ; CMOV-32: mov.s $f0, $f14
>
> - ; SEL-32: mtc1 $6, $f0
> + ; SEL-32: sll $[[T0:[0-9]+]], $6, 31
> + ; SEL-32: sra $[[T1:[0-9]+]], $[[T0]], 31
> + ; SEL-32: mtc1 $[[T1]], $f0
> ; SEL-32: sel.s $f0, $f14, $f12
>
> ; CMOV-64: andi $[[T0:[0-9]+]], $6, 1
> ; CMOV-64: movn.s $f13, $f12, $[[T0]]
> ; CMOV-64: mov.s $f0, $f13
>
> - ; SEL-64: mtc1 $6, $f0
> + ; SEL-64: dsll $[[T0:[0-9]+]], $6, 63
> + ; SEL-64: dsra $[[T1:[0-9]+]], $[[T0]], 63
> + ; SEL-64: dmtc1 $[[T1]], $f0
> ; SEL-64: sel.s $f0, $f13, $f12
> %r = select i1 %s, float %x, float %y
> ret float %r
>
> Modified: llvm/trunk/test/CodeGen/Mips/llvm-ir/select-int.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/llvm-ir/select-int.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/llvm-ir/select-int.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/llvm-ir/select-int.ll Tue Mar 1 04:08:01 2016
> @@ -29,8 +29,7 @@
> ; RUN: llc < %s -march=mips64 -mcpu=mips64r6 | FileCheck %s \
> ; RUN: -check-prefix=ALL -check-prefix=SEL -check-prefix=SEL-64
>
> -define signext i1 @tst_select_i1_i1(i1 signext %s,
> - i1 signext %x, i1 signext %y) {
> +define signext i1 @tst_select_i1_i1(i1 %s, i1 signext %x, i1 signext %y) {
> entry:
> ; ALL-LABEL: tst_select_i1_i1:
>
> @@ -54,8 +53,7 @@ entry:
> ret i1 %r
> }
>
> -define signext i8 @tst_select_i1_i8(i1 signext %s,
> - i8 signext %x, i8 signext %y) {
> +define signext i8 @tst_select_i1_i8(i1 %s, i8 signext %x, i8 signext %y) {
> entry:
> ; ALL-LABEL: tst_select_i1_i8:
>
> @@ -79,8 +77,7 @@ entry:
> ret i8 %r
> }
>
> -define signext i32 @tst_select_i1_i32(i1 signext %s,
> - i32 signext %x, i32 signext %y) {
> +define signext i32 @tst_select_i1_i32(i1 %s, i32 signext %x, i32 signext %y) {
> entry:
> ; ALL-LABEL: tst_select_i1_i32:
>
> @@ -104,8 +101,7 @@ entry:
> ret i32 %r
> }
>
> -define signext i64 @tst_select_i1_i64(i1 signext %s,
> - i64 signext %x, i64 signext %y) {
> +define signext i64 @tst_select_i1_i64(i1 %s, i64 signext %x, i64 signext %y) {
> entry:
> ; ALL-LABEL: tst_select_i1_i64:
>
> @@ -152,8 +148,6 @@ entry:
> ; CMOV-64: move $2, $6
>
> ; SEL-64: andi $[[T0:[0-9]+]], $4, 1
> - ; FIXME: This shift is redundant
> - ; SEL-64: sll $[[T0]], $[[T0]], 0
> ; SEL-64: seleqz $[[T1:[0-9]+]], $6, $[[T0]]
> ; SEL-64: selnez $[[T0]], $5, $[[T0]]
> ; SEL-64: or $2, $[[T0]], $[[T1]]
>
> Modified: llvm/trunk/test/CodeGen/Mips/octeon.ll
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/octeon.ll?rev=262316&r1=262315&r2=262316&view=diff
> ==============================================================================
> --- llvm/trunk/test/CodeGen/Mips/octeon.ll (original)
> +++ llvm/trunk/test/CodeGen/Mips/octeon.ll Tue Mar 1 04:08:01 2016
> @@ -32,10 +32,8 @@ entry:
> ; OCTEON: jr $ra
> ; OCTEON: seq $2, $4, $5
> ; MIPS64: xor $[[T0:[0-9]+]], $4, $5
> -; MIPS64: sltiu $[[T1:[0-9]+]], $[[T0]], 1
> -; MIPS64: dsll $[[T2:[0-9]+]], $[[T1]], 32
> ; MIPS64: jr $ra
> -; MIPS64: dsrl $2, $[[T2]], 32
> +; MIPS64: sltiu $2, $[[T0]], 1
> %res = icmp eq i64 %a, %b
> %res2 = zext i1 %res to i64
> ret i64 %res2
> @@ -48,10 +46,8 @@ entry:
> ; OCTEON: seqi $2, $4, 42
> ; MIPS64: daddiu $[[T0:[0-9]+]], $zero, 42
> ; MIPS64: xor $[[T1:[0-9]+]], $4, $[[T0]]
> -; MIPS64: sltiu $[[T2:[0-9]+]], $[[T1]], 1
> -; MIPS64: dsll $[[T3:[0-9]+]], $[[T2]], 32
> ; MIPS64: jr $ra
> -; MIPS64: dsrl $2, $[[T3]], 32
> +; MIPS64: sltiu $2, $[[T1]], 1
> %res = icmp eq i64 %a, 42
> %res2 = zext i1 %res to i64
> ret i64 %res2
> @@ -63,10 +59,8 @@ entry:
> ; OCTEON: jr $ra
> ; OCTEON: sne $2, $4, $5
> ; MIPS64: xor $[[T0:[0-9]+]], $4, $5
> -; MIPS64: sltu $[[T1:[0-9]+]], $zero, $[[T0]]
> -; MIPS64: dsll $[[T2:[0-9]+]], $[[T1]], 32
> ; MIPS64: jr $ra
> -; MIPS64: dsrl $2, $[[T2]], 32
> +; MIPS64: sltu $2, $zero, $[[T0]]
> %res = icmp ne i64 %a, %b
> %res2 = zext i1 %res to i64
> ret i64 %res2
> @@ -79,10 +73,8 @@ entry:
> ; OCTEON: snei $2, $4, 42
> ; MIPS64: daddiu $[[T0:[0-9]+]], $zero, 42
> ; MIPS64: xor $[[T1:[0-9]+]], $4, $[[T0]]
> -; MIPS64: sltu $[[T2:[0-9]+]], $zero, $[[T1]]
> -; MIPS64: dsll $[[T3:[0-9]+]], $[[T2]], 32
> ; MIPS64: jr $ra
> -; MIPS64: dsrl $2, $[[T3]], 32
> +; MIPS64: sltu $2, $zero, $[[T1]]
> %res = icmp ne i64 %a, 42
> %res2 = zext i1 %res to i64
> ret i64 %res2
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list