1139f7f9bSDimitry Andric //===-------- X86PadShortFunction.cpp - pad short functions -----------===//
2139f7f9bSDimitry Andric //
3139f7f9bSDimitry Andric // The LLVM Compiler Infrastructure
4139f7f9bSDimitry Andric //
5139f7f9bSDimitry Andric // This file is distributed under the University of Illinois Open Source
6139f7f9bSDimitry Andric // License. See LICENSE.TXT for details.
7139f7f9bSDimitry Andric //
8139f7f9bSDimitry Andric //===----------------------------------------------------------------------===//
9139f7f9bSDimitry Andric //
10139f7f9bSDimitry Andric // This file defines the pass which will pad short functions to prevent
11139f7f9bSDimitry Andric // a stall if a function returns before the return address is ready. This
12139f7f9bSDimitry Andric // is needed for some Intel Atom processors.
13139f7f9bSDimitry Andric //
14139f7f9bSDimitry Andric //===----------------------------------------------------------------------===//
15139f7f9bSDimitry Andric
16139f7f9bSDimitry Andric
17139f7f9bSDimitry Andric #include "X86.h"
18139f7f9bSDimitry Andric #include "X86InstrInfo.h"
1991bc56edSDimitry Andric #include "X86Subtarget.h"
20139f7f9bSDimitry Andric #include "llvm/ADT/Statistic.h"
21139f7f9bSDimitry Andric #include "llvm/CodeGen/MachineFunctionPass.h"
22139f7f9bSDimitry Andric #include "llvm/CodeGen/MachineInstrBuilder.h"
23139f7f9bSDimitry Andric #include "llvm/CodeGen/Passes.h"
24*4ba319b5SDimitry Andric #include "llvm/CodeGen/TargetSchedule.h"
25139f7f9bSDimitry Andric #include "llvm/IR/Function.h"
26139f7f9bSDimitry Andric #include "llvm/Support/Debug.h"
27139f7f9bSDimitry Andric #include "llvm/Support/raw_ostream.h"
28139f7f9bSDimitry Andric
29139f7f9bSDimitry Andric using namespace llvm;
30139f7f9bSDimitry Andric
3191bc56edSDimitry Andric #define DEBUG_TYPE "x86-pad-short-functions"
3291bc56edSDimitry Andric
33139f7f9bSDimitry Andric STATISTIC(NumBBsPadded, "Number of basic blocks padded");
34139f7f9bSDimitry Andric
35139f7f9bSDimitry Andric namespace {
36139f7f9bSDimitry Andric struct VisitedBBInfo {
37139f7f9bSDimitry Andric // HasReturn - Whether the BB contains a return instruction
38139f7f9bSDimitry Andric bool HasReturn;
39139f7f9bSDimitry Andric
40139f7f9bSDimitry Andric // Cycles - Number of cycles until return if HasReturn is true, otherwise
41139f7f9bSDimitry Andric // number of cycles until end of the BB
42139f7f9bSDimitry Andric unsigned int Cycles;
43139f7f9bSDimitry Andric
VisitedBBInfo__anon43d0aa510111::VisitedBBInfo44139f7f9bSDimitry Andric VisitedBBInfo() : HasReturn(false), Cycles(0) {}
VisitedBBInfo__anon43d0aa510111::VisitedBBInfo45139f7f9bSDimitry Andric VisitedBBInfo(bool HasReturn, unsigned int Cycles)
46139f7f9bSDimitry Andric : HasReturn(HasReturn), Cycles(Cycles) {}
47139f7f9bSDimitry Andric };
48139f7f9bSDimitry Andric
49139f7f9bSDimitry Andric struct PadShortFunc : public MachineFunctionPass {
50139f7f9bSDimitry Andric static char ID;
PadShortFunc__anon43d0aa510111::PadShortFunc51139f7f9bSDimitry Andric PadShortFunc() : MachineFunctionPass(ID)
52*4ba319b5SDimitry Andric , Threshold(4) {}
53139f7f9bSDimitry Andric
5491bc56edSDimitry Andric bool runOnMachineFunction(MachineFunction &MF) override;
55139f7f9bSDimitry Andric
getRequiredProperties__anon43d0aa510111::PadShortFunc563ca95b02SDimitry Andric MachineFunctionProperties getRequiredProperties() const override {
573ca95b02SDimitry Andric return MachineFunctionProperties().set(
58d88c1a5aSDimitry Andric MachineFunctionProperties::Property::NoVRegs);
593ca95b02SDimitry Andric }
603ca95b02SDimitry Andric
getPassName__anon43d0aa510111::PadShortFunc61d88c1a5aSDimitry Andric StringRef getPassName() const override {
62139f7f9bSDimitry Andric return "X86 Atom pad short functions";
63139f7f9bSDimitry Andric }
64139f7f9bSDimitry Andric
65139f7f9bSDimitry Andric private:
66139f7f9bSDimitry Andric void findReturns(MachineBasicBlock *MBB,
67139f7f9bSDimitry Andric unsigned int Cycles = 0);
68139f7f9bSDimitry Andric
69139f7f9bSDimitry Andric bool cyclesUntilReturn(MachineBasicBlock *MBB,
70139f7f9bSDimitry Andric unsigned int &Cycles);
71139f7f9bSDimitry Andric
72139f7f9bSDimitry Andric void addPadding(MachineBasicBlock *MBB,
73139f7f9bSDimitry Andric MachineBasicBlock::iterator &MBBI,
74139f7f9bSDimitry Andric unsigned int NOOPsToAdd);
75139f7f9bSDimitry Andric
76139f7f9bSDimitry Andric const unsigned int Threshold;
77139f7f9bSDimitry Andric
78139f7f9bSDimitry Andric // ReturnBBs - Maps basic blocks that return to the minimum number of
79139f7f9bSDimitry Andric // cycles until the return, starting from the entry block.
80139f7f9bSDimitry Andric DenseMap<MachineBasicBlock*, unsigned int> ReturnBBs;
81139f7f9bSDimitry Andric
82139f7f9bSDimitry Andric // VisitedBBs - Cache of previously visited BBs.
83139f7f9bSDimitry Andric DenseMap<MachineBasicBlock*, VisitedBBInfo> VisitedBBs;
84139f7f9bSDimitry Andric
85*4ba319b5SDimitry Andric TargetSchedModel TSM;
86139f7f9bSDimitry Andric };
87139f7f9bSDimitry Andric
88139f7f9bSDimitry Andric char PadShortFunc::ID = 0;
893dac3a9bSDimitry Andric }
90139f7f9bSDimitry Andric
createX86PadShortFunctions()91139f7f9bSDimitry Andric FunctionPass *llvm::createX86PadShortFunctions() {
92139f7f9bSDimitry Andric return new PadShortFunc();
93139f7f9bSDimitry Andric }
94139f7f9bSDimitry Andric
95139f7f9bSDimitry Andric /// runOnMachineFunction - Loop over all of the basic blocks, inserting
96139f7f9bSDimitry Andric /// NOOP instructions before early exits.
runOnMachineFunction(MachineFunction & MF)97139f7f9bSDimitry Andric bool PadShortFunc::runOnMachineFunction(MachineFunction &MF) {
982cab237bSDimitry Andric if (skipFunction(MF.getFunction()))
993ca95b02SDimitry Andric return false;
1003ca95b02SDimitry Andric
101*4ba319b5SDimitry Andric if (MF.getFunction().optForSize())
10291bc56edSDimitry Andric return false;
10391bc56edSDimitry Andric
104*4ba319b5SDimitry Andric if (!MF.getSubtarget<X86Subtarget>().padShortFunctions())
105*4ba319b5SDimitry Andric return false;
106*4ba319b5SDimitry Andric
107*4ba319b5SDimitry Andric TSM.init(&MF.getSubtarget());
108139f7f9bSDimitry Andric
109139f7f9bSDimitry Andric // Search through basic blocks and mark the ones that have early returns
110139f7f9bSDimitry Andric ReturnBBs.clear();
111139f7f9bSDimitry Andric VisitedBBs.clear();
1127d523365SDimitry Andric findReturns(&MF.front());
113139f7f9bSDimitry Andric
114139f7f9bSDimitry Andric bool MadeChange = false;
115139f7f9bSDimitry Andric
116139f7f9bSDimitry Andric MachineBasicBlock *MBB;
117139f7f9bSDimitry Andric unsigned int Cycles = 0;
118139f7f9bSDimitry Andric
119139f7f9bSDimitry Andric // Pad the identified basic blocks with NOOPs
120139f7f9bSDimitry Andric for (DenseMap<MachineBasicBlock*, unsigned int>::iterator I = ReturnBBs.begin();
121139f7f9bSDimitry Andric I != ReturnBBs.end(); ++I) {
122139f7f9bSDimitry Andric MBB = I->first;
123139f7f9bSDimitry Andric Cycles = I->second;
124139f7f9bSDimitry Andric
125139f7f9bSDimitry Andric if (Cycles < Threshold) {
126139f7f9bSDimitry Andric // BB ends in a return. Skip over any DBG_VALUE instructions
127139f7f9bSDimitry Andric // trailing the terminator.
128139f7f9bSDimitry Andric assert(MBB->size() > 0 &&
129139f7f9bSDimitry Andric "Basic block should contain at least a RET but is empty");
130139f7f9bSDimitry Andric MachineBasicBlock::iterator ReturnLoc = --MBB->end();
131139f7f9bSDimitry Andric
132*4ba319b5SDimitry Andric while (ReturnLoc->isDebugInstr())
133139f7f9bSDimitry Andric --ReturnLoc;
134139f7f9bSDimitry Andric assert(ReturnLoc->isReturn() && !ReturnLoc->isCall() &&
135139f7f9bSDimitry Andric "Basic block does not end with RET");
136139f7f9bSDimitry Andric
137139f7f9bSDimitry Andric addPadding(MBB, ReturnLoc, Threshold - Cycles);
138139f7f9bSDimitry Andric NumBBsPadded++;
139139f7f9bSDimitry Andric MadeChange = true;
140139f7f9bSDimitry Andric }
141139f7f9bSDimitry Andric }
142139f7f9bSDimitry Andric
143139f7f9bSDimitry Andric return MadeChange;
144139f7f9bSDimitry Andric }
145139f7f9bSDimitry Andric
146139f7f9bSDimitry Andric /// findReturn - Starting at MBB, follow control flow and add all
147139f7f9bSDimitry Andric /// basic blocks that contain a return to ReturnBBs.
findReturns(MachineBasicBlock * MBB,unsigned int Cycles)148139f7f9bSDimitry Andric void PadShortFunc::findReturns(MachineBasicBlock *MBB, unsigned int Cycles) {
149139f7f9bSDimitry Andric // If this BB has a return, note how many cycles it takes to get there.
150139f7f9bSDimitry Andric bool hasReturn = cyclesUntilReturn(MBB, Cycles);
151139f7f9bSDimitry Andric if (Cycles >= Threshold)
152139f7f9bSDimitry Andric return;
153139f7f9bSDimitry Andric
154139f7f9bSDimitry Andric if (hasReturn) {
155139f7f9bSDimitry Andric ReturnBBs[MBB] = std::max(ReturnBBs[MBB], Cycles);
156139f7f9bSDimitry Andric return;
157139f7f9bSDimitry Andric }
158139f7f9bSDimitry Andric
159139f7f9bSDimitry Andric // Follow branches in BB and look for returns
160139f7f9bSDimitry Andric for (MachineBasicBlock::succ_iterator I = MBB->succ_begin();
161139f7f9bSDimitry Andric I != MBB->succ_end(); ++I) {
162139f7f9bSDimitry Andric if (*I == MBB)
163139f7f9bSDimitry Andric continue;
164139f7f9bSDimitry Andric findReturns(*I, Cycles);
165139f7f9bSDimitry Andric }
166139f7f9bSDimitry Andric }
167139f7f9bSDimitry Andric
168139f7f9bSDimitry Andric /// cyclesUntilReturn - return true if the MBB has a return instruction,
169139f7f9bSDimitry Andric /// and return false otherwise.
170139f7f9bSDimitry Andric /// Cycles will be incremented by the number of cycles taken to reach the
171139f7f9bSDimitry Andric /// return or the end of the BB, whichever occurs first.
cyclesUntilReturn(MachineBasicBlock * MBB,unsigned int & Cycles)172139f7f9bSDimitry Andric bool PadShortFunc::cyclesUntilReturn(MachineBasicBlock *MBB,
173139f7f9bSDimitry Andric unsigned int &Cycles) {
174139f7f9bSDimitry Andric // Return cached result if BB was previously visited
175139f7f9bSDimitry Andric DenseMap<MachineBasicBlock*, VisitedBBInfo>::iterator it
176139f7f9bSDimitry Andric = VisitedBBs.find(MBB);
177139f7f9bSDimitry Andric if (it != VisitedBBs.end()) {
178139f7f9bSDimitry Andric VisitedBBInfo BBInfo = it->second;
179139f7f9bSDimitry Andric Cycles += BBInfo.Cycles;
180139f7f9bSDimitry Andric return BBInfo.HasReturn;
181139f7f9bSDimitry Andric }
182139f7f9bSDimitry Andric
183139f7f9bSDimitry Andric unsigned int CyclesToEnd = 0;
184139f7f9bSDimitry Andric
1853ca95b02SDimitry Andric for (MachineInstr &MI : *MBB) {
186139f7f9bSDimitry Andric // Mark basic blocks with a return instruction. Calls to other
187139f7f9bSDimitry Andric // functions do not count because the called function will be padded,
188139f7f9bSDimitry Andric // if necessary.
1893ca95b02SDimitry Andric if (MI.isReturn() && !MI.isCall()) {
190139f7f9bSDimitry Andric VisitedBBs[MBB] = VisitedBBInfo(true, CyclesToEnd);
191139f7f9bSDimitry Andric Cycles += CyclesToEnd;
192139f7f9bSDimitry Andric return true;
193139f7f9bSDimitry Andric }
194139f7f9bSDimitry Andric
195*4ba319b5SDimitry Andric CyclesToEnd += TSM.computeInstrLatency(&MI);
196139f7f9bSDimitry Andric }
197139f7f9bSDimitry Andric
198139f7f9bSDimitry Andric VisitedBBs[MBB] = VisitedBBInfo(false, CyclesToEnd);
199139f7f9bSDimitry Andric Cycles += CyclesToEnd;
200139f7f9bSDimitry Andric return false;
201139f7f9bSDimitry Andric }
202139f7f9bSDimitry Andric
203139f7f9bSDimitry Andric /// addPadding - Add the given number of NOOP instructions to the function
204139f7f9bSDimitry Andric /// just prior to the return at MBBI
addPadding(MachineBasicBlock * MBB,MachineBasicBlock::iterator & MBBI,unsigned int NOOPsToAdd)205139f7f9bSDimitry Andric void PadShortFunc::addPadding(MachineBasicBlock *MBB,
206139f7f9bSDimitry Andric MachineBasicBlock::iterator &MBBI,
207139f7f9bSDimitry Andric unsigned int NOOPsToAdd) {
208139f7f9bSDimitry Andric DebugLoc DL = MBBI->getDebugLoc();
209*4ba319b5SDimitry Andric unsigned IssueWidth = TSM.getIssueWidth();
210139f7f9bSDimitry Andric
211*4ba319b5SDimitry Andric for (unsigned i = 0, e = IssueWidth * NOOPsToAdd; i != e; ++i)
212*4ba319b5SDimitry Andric BuildMI(*MBB, MBBI, DL, TSM.getInstrInfo()->get(X86::NOOP));
213139f7f9bSDimitry Andric }
214