[PATCH] D68375: [llvm-exegesis][NFC] Rename ExegesisTarget::decrementLoopCounterAndLoop()
Clement Courbet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 3 00:48:27 PDT 2019
courbet created this revision.
courbet added a reviewer: gchatelet.
Herald added a subscriber: tschuett.
Herald added a project: LLVM.
To decrementLoopCounterAndJump, and explicitely take the jump target.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D68375
Files:
llvm/tools/llvm-exegesis/lib/SnippetRepetitor.cpp
llvm/tools/llvm-exegesis/lib/Target.h
llvm/tools/llvm-exegesis/lib/X86/Target.cpp
Index: llvm/tools/llvm-exegesis/lib/X86/Target.cpp
===================================================================
--- llvm/tools/llvm-exegesis/lib/X86/Target.cpp
+++ llvm/tools/llvm-exegesis/lib/X86/Target.cpp
@@ -448,7 +448,8 @@
void fillMemoryOperands(InstructionTemplate &IT, unsigned Reg,
unsigned Offset) const override;
- void decrementLoopCounterAndLoop(MachineBasicBlock &MBB,
+ void decrementLoopCounterAndJump(MachineBasicBlock &MBB,
+ MachineBasicBlock &TargetMBB,
const llvm::MCInstrInfo &MII) const override;
std::vector<llvm::MCInst> setRegTo(const llvm::MCSubtargetInfo &STI,
@@ -558,14 +559,15 @@
SetOp(MemOpIdx + 4, MCOperand::createReg(0)); // Segment
}
-void ExegesisX86Target::decrementLoopCounterAndLoop(
- MachineBasicBlock &MBB, const llvm::MCInstrInfo &MII) const {
+void ExegesisX86Target::decrementLoopCounterAndJump(
+ MachineBasicBlock &MBB, MachineBasicBlock &TargetMBB,
+ const llvm::MCInstrInfo &MII) const {
BuildMI(&MBB, DebugLoc(), MII.get(X86::ADD64ri8))
.addDef(kLoopCounterReg)
.addUse(kLoopCounterReg)
.addImm(-1);
BuildMI(&MBB, DebugLoc(), MII.get(X86::JCC_1))
- .addMBB(&MBB)
+ .addMBB(&TargetMBB)
.addImm(X86::COND_NE);
}
Index: llvm/tools/llvm-exegesis/lib/Target.h
===================================================================
--- llvm/tools/llvm-exegesis/lib/Target.h
+++ llvm/tools/llvm-exegesis/lib/Target.h
@@ -95,7 +95,8 @@
}
// Adds the code to decrement the loop counter and
- virtual void decrementLoopCounterAndLoop(MachineBasicBlock &MBB,
+ virtual void decrementLoopCounterAndJump(MachineBasicBlock &MBB,
+ MachineBasicBlock &TargetMBB,
const llvm::MCInstrInfo &MII) const {
llvm_unreachable("decrementLoopCounterAndBranch() requires "
"getLoopCounterRegister() > 0");
Index: llvm/tools/llvm-exegesis/lib/SnippetRepetitor.cpp
===================================================================
--- llvm/tools/llvm-exegesis/lib/SnippetRepetitor.cpp
+++ llvm/tools/llvm-exegesis/lib/SnippetRepetitor.cpp
@@ -79,7 +79,8 @@
for (const auto &LiveIn : Entry.MBB->liveins())
Loop.MBB->addLiveIn(LiveIn);
Loop.addInstructions(Instructions);
- ET.decrementLoopCounterAndLoop(*Loop.MBB, State.getInstrInfo());
+ ET.decrementLoopCounterAndJump(*Loop.MBB, *Loop.MBB,
+ State.getInstrInfo());
// Set up the exit basic block.
Loop.MBB->addSuccessor(Exit.MBB, llvm::BranchProbability::getZero());
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68375.222971.patch
Type: text/x-patch
Size: 2731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191003/81a90503/attachment.bin>
More information about the llvm-commits
mailing list