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