From 2ee218f3653fcfd135fc5087a698424f90a9a499 Mon Sep 17 00:00:00 2001 From: Clement Courbet Date: Fri, 24 Jan 2020 11:42:27 +0100 Subject: [PATCH] [llvm-exegesis][NFC] Simplify code. Summary: What we're redoing already exists in the X86 backend, it's called `X86II::getOperandBias`. Reviewers: gchatelet Subscribers: tschuett, mstojanovic, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D73340 --- llvm/tools/llvm-exegesis/lib/X86/Target.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/llvm/tools/llvm-exegesis/lib/X86/Target.cpp b/llvm/tools/llvm-exegesis/lib/X86/Target.cpp index a76e590cfa2..4daf42201a1 100644 --- a/llvm/tools/llvm-exegesis/lib/X86/Target.cpp +++ b/llvm/tools/llvm-exegesis/lib/X86/Target.cpp @@ -673,12 +673,7 @@ void ExegesisX86Target::fillMemoryOperands(InstructionTemplate &IT, int MemOpIdx = X86II::getMemoryOperandNo(IT.getInstr().Description.TSFlags); assert(MemOpIdx >= 0 && "invalid memory operand index"); // getMemoryOperandNo() ignores tied operands, so we have to add them back. - for (unsigned I = 0; I <= static_cast(MemOpIdx); ++I) { - const auto &Op = IT.getInstr().Operands[I]; - if (Op.isTied() && Op.getTiedToIndex() < I) { - ++MemOpIdx; - } - } + MemOpIdx += X86II::getOperandBias(IT.getInstr().Description); setMemOp(IT, MemOpIdx + 0, MCOperand::createReg(Reg)); // BaseReg setMemOp(IT, MemOpIdx + 1, MCOperand::createImm(1)); // ScaleAmt setMemOp(IT, MemOpIdx + 2, MCOperand::createReg(0)); // IndexReg