From 35636281c7ab6eb128b4ced6bf7ae0b6b8458dd2 Mon Sep 17 00:00:00 2001 From: Jim Grosbach Date: Wed, 6 Oct 2010 21:22:32 +0000 Subject: [PATCH] Now that VDUPfqf and VDUPfdfare properly pseudos, kill the no-longer-needed "lane" operand modifier. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@115843 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/ARM/ARMAsmPrinter.cpp | 12 ++----- lib/Target/ARM/InstPrinter/ARMInstPrinter.cpp | 31 +------------------ 2 files changed, 3 insertions(+), 40 deletions(-) diff --git a/lib/Target/ARM/ARMAsmPrinter.cpp b/lib/Target/ARM/ARMAsmPrinter.cpp index ecd9567fae7..b9d38d148a0 100644 --- a/lib/Target/ARM/ARMAsmPrinter.cpp +++ b/lib/Target/ARM/ARMAsmPrinter.cpp @@ -236,16 +236,8 @@ void ARMAsmPrinter::printOperand(const MachineInstr *MI, int OpNum, case MachineOperand::MO_Register: { unsigned Reg = MO.getReg(); assert(TargetRegisterInfo::isPhysicalRegister(Reg)); - if (Modifier && strcmp(Modifier, "lane") == 0) { - unsigned RegNum = getARMRegisterNumbering(Reg); - unsigned DReg = - TM.getRegisterInfo()->getMatchingSuperReg(Reg, - RegNum & 1 ? ARM::ssub_1 : ARM::ssub_0, &ARM::DPR_VFP2RegClass); - O << ARMInstPrinter::getRegisterName(DReg) << '[' << (RegNum & 1) << ']'; - } else { - assert(!MO.getSubReg() && "Subregs should be eliminated!"); - O << ARMInstPrinter::getRegisterName(Reg); - } + assert(!MO.getSubReg() && "Subregs should be eliminated!"); + O << ARMInstPrinter::getRegisterName(Reg); break; } case MachineOperand::MO_Immediate: { diff --git a/lib/Target/ARM/InstPrinter/ARMInstPrinter.cpp b/lib/Target/ARM/InstPrinter/ARMInstPrinter.cpp index f97bf0b3d6a..8c16d3cb80d 100644 --- a/lib/Target/ARM/InstPrinter/ARMInstPrinter.cpp +++ b/lib/Target/ARM/InstPrinter/ARMInstPrinter.cpp @@ -24,29 +24,6 @@ using namespace llvm; #include "ARMGenAsmWriter.inc" -static unsigned getDPRSuperRegForSPR(unsigned Reg) { - switch (Reg) { - default: - assert(0 && "Unexpected register enum"); - case ARM::S0: case ARM::S1: return ARM::D0; - case ARM::S2: case ARM::S3: return ARM::D1; - case ARM::S4: case ARM::S5: return ARM::D2; - case ARM::S6: case ARM::S7: return ARM::D3; - case ARM::S8: case ARM::S9: return ARM::D4; - case ARM::S10: case ARM::S11: return ARM::D5; - case ARM::S12: case ARM::S13: return ARM::D6; - case ARM::S14: case ARM::S15: return ARM::D7; - case ARM::S16: case ARM::S17: return ARM::D8; - case ARM::S18: case ARM::S19: return ARM::D9; - case ARM::S20: case ARM::S21: return ARM::D10; - case ARM::S22: case ARM::S23: return ARM::D11; - case ARM::S24: case ARM::S25: return ARM::D12; - case ARM::S26: case ARM::S27: return ARM::D13; - case ARM::S28: case ARM::S29: return ARM::D14; - case ARM::S30: case ARM::S31: return ARM::D15; - } -} - void ARMInstPrinter::printInst(const MCInst *MI, raw_ostream &O) { // Check for MOVs and print canonical forms, instead. if (MI->getOpcode() == ARM::MOVs) { @@ -137,13 +114,7 @@ void ARMInstPrinter::printOperand(const MCInst *MI, unsigned OpNo, const MCOperand &Op = MI->getOperand(OpNo); if (Op.isReg()) { unsigned Reg = Op.getReg(); - if (Modifier && strcmp(Modifier, "lane") == 0) { - unsigned RegNum = getARMRegisterNumbering(Reg); - unsigned DReg = getDPRSuperRegForSPR(Reg); - O << getRegisterName(DReg) << '[' << (RegNum & 1) << ']'; - } else { - O << getRegisterName(Reg); - } + O << getRegisterName(Reg); } else if (Op.isImm()) { assert((Modifier && !strcmp(Modifier, "call")) || ((Modifier == 0 || Modifier[0] == 0) && "No modifiers supported"));