blob: 73144cb28f15a7cb8f4bfc9e4ac3b2ac87337e20 [file] [log] [blame]
Misha Brukman91b5ca82004-07-26 18:45:48 +00001//===-- X86FloatingPoint.cpp - Floating point Reg -> Stack converter ------===//
Misha Brukman0e0a7a452005-04-21 23:38:14 +00002//
John Criswellb576c942003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
5// This file was developed by the LLVM research group and is distributed under
6// the University of Illinois Open Source License. See LICENSE.TXT for details.
Misha Brukman0e0a7a452005-04-21 23:38:14 +00007//
John Criswellb576c942003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattnera960d952003-01-13 01:01:59 +00009//
10// This file defines the pass which converts floating point instructions from
Chris Lattner847df252004-01-30 22:25:18 +000011// virtual registers into register stack instructions. This pass uses live
12// variable information to indicate where the FPn registers are used and their
13// lifetimes.
14//
15// This pass is hampered by the lack of decent CFG manipulation routines for
16// machine code. In particular, this wants to be able to split critical edges
17// as necessary, traverse the machine basic block CFG in depth-first order, and
18// allow there to be multiple machine basic blocks for each LLVM basicblock
19// (needed for critical edge splitting).
20//
21// In particular, this pass currently barfs on critical edges. Because of this,
22// it requires the instruction selector to insert FP_REG_KILL instructions on
23// the exits of any basic block that has critical edges going from it, or which
24// branch to a critical basic block.
25//
26// FIXME: this is not implemented yet. The stackifier pass only works on local
27// basic blocks.
Chris Lattnera960d952003-01-13 01:01:59 +000028//
29//===----------------------------------------------------------------------===//
30
Chris Lattnercb533582003-08-03 21:14:38 +000031#define DEBUG_TYPE "fp"
Chris Lattnera960d952003-01-13 01:01:59 +000032#include "X86.h"
33#include "X86InstrInfo.h"
34#include "llvm/CodeGen/MachineFunctionPass.h"
35#include "llvm/CodeGen/MachineInstrBuilder.h"
36#include "llvm/CodeGen/LiveVariables.h"
Alkis Evlogimenos359b65f2003-12-13 05:36:22 +000037#include "llvm/CodeGen/Passes.h"
Chris Lattner3501fea2003-01-14 22:00:31 +000038#include "llvm/Target/TargetInstrInfo.h"
Chris Lattnera960d952003-01-13 01:01:59 +000039#include "llvm/Target/TargetMachine.h"
Reid Spencer551ccae2004-09-01 22:55:40 +000040#include "llvm/Support/Debug.h"
41#include "llvm/ADT/DepthFirstIterator.h"
42#include "llvm/ADT/Statistic.h"
43#include "llvm/ADT/STLExtras.h"
Chris Lattnera960d952003-01-13 01:01:59 +000044#include <algorithm>
Chris Lattner2c2c6c62006-01-22 23:41:00 +000045#include <iostream>
Chris Lattner847df252004-01-30 22:25:18 +000046#include <set>
Chris Lattnerf2e49d42003-12-20 09:58:55 +000047using namespace llvm;
Brian Gaeked0fde302003-11-11 22:41:34 +000048
Chris Lattnera960d952003-01-13 01:01:59 +000049namespace {
50 Statistic<> NumFXCH("x86-codegen", "Number of fxch instructions inserted");
51 Statistic<> NumFP ("x86-codegen", "Number of floating point instructions");
52
53 struct FPS : public MachineFunctionPass {
54 virtual bool runOnMachineFunction(MachineFunction &MF);
55
56 virtual const char *getPassName() const { return "X86 FP Stackifier"; }
57
58 virtual void getAnalysisUsage(AnalysisUsage &AU) const {
59 AU.addRequired<LiveVariables>();
60 MachineFunctionPass::getAnalysisUsage(AU);
61 }
62 private:
63 LiveVariables *LV; // Live variable info for current function...
64 MachineBasicBlock *MBB; // Current basic block
65 unsigned Stack[8]; // FP<n> Registers in each stack slot...
66 unsigned RegMap[8]; // Track which stack slot contains each register
67 unsigned StackTop; // The current top of the FP stack.
68
69 void dumpStack() const {
70 std::cerr << "Stack contents:";
71 for (unsigned i = 0; i != StackTop; ++i) {
Misha Brukman0e0a7a452005-04-21 23:38:14 +000072 std::cerr << " FP" << Stack[i];
73 assert(RegMap[Stack[i]] == i && "Stack[] doesn't match RegMap[]!");
Chris Lattnera960d952003-01-13 01:01:59 +000074 }
75 std::cerr << "\n";
76 }
77 private:
78 // getSlot - Return the stack slot number a particular register number is
79 // in...
80 unsigned getSlot(unsigned RegNo) const {
81 assert(RegNo < 8 && "Regno out of range!");
82 return RegMap[RegNo];
83 }
84
85 // getStackEntry - Return the X86::FP<n> register in register ST(i)
86 unsigned getStackEntry(unsigned STi) const {
87 assert(STi < StackTop && "Access past stack top!");
88 return Stack[StackTop-1-STi];
89 }
90
91 // getSTReg - Return the X86::ST(i) register which contains the specified
92 // FP<RegNo> register
93 unsigned getSTReg(unsigned RegNo) const {
Brian Gaeked0fde302003-11-11 22:41:34 +000094 return StackTop - 1 - getSlot(RegNo) + llvm::X86::ST0;
Chris Lattnera960d952003-01-13 01:01:59 +000095 }
96
Chris Lattner4a06f352004-02-02 19:23:15 +000097 // pushReg - Push the specified FP<n> register onto the stack
Chris Lattnera960d952003-01-13 01:01:59 +000098 void pushReg(unsigned Reg) {
99 assert(Reg < 8 && "Register number out of range!");
100 assert(StackTop < 8 && "Stack overflow!");
101 Stack[StackTop] = Reg;
102 RegMap[Reg] = StackTop++;
103 }
104
105 bool isAtTop(unsigned RegNo) const { return getSlot(RegNo) == StackTop-1; }
106 void moveToTop(unsigned RegNo, MachineBasicBlock::iterator &I) {
107 if (!isAtTop(RegNo)) {
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000108 unsigned Slot = getSlot(RegNo);
109 unsigned STReg = getSTReg(RegNo);
110 unsigned RegOnTop = getStackEntry(0);
Chris Lattnera960d952003-01-13 01:01:59 +0000111
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000112 // Swap the slots the regs are in
113 std::swap(RegMap[RegNo], RegMap[RegOnTop]);
Chris Lattnera960d952003-01-13 01:01:59 +0000114
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000115 // Swap stack slot contents
116 assert(RegMap[RegOnTop] < StackTop);
117 std::swap(Stack[RegMap[RegOnTop]], Stack[StackTop-1]);
Chris Lattnera960d952003-01-13 01:01:59 +0000118
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000119 // Emit an fxch to update the runtime processors version of the state
120 BuildMI(*MBB, I, X86::FXCH, 1).addReg(STReg);
121 NumFXCH++;
Chris Lattnera960d952003-01-13 01:01:59 +0000122 }
123 }
124
Chris Lattner0526f012004-04-01 04:06:09 +0000125 void duplicateToTop(unsigned RegNo, unsigned AsReg, MachineInstr *I) {
Chris Lattnera960d952003-01-13 01:01:59 +0000126 unsigned STReg = getSTReg(RegNo);
127 pushReg(AsReg); // New register on top of stack
128
Chris Lattner0526f012004-04-01 04:06:09 +0000129 BuildMI(*MBB, I, X86::FLDrr, 1).addReg(STReg);
Chris Lattnera960d952003-01-13 01:01:59 +0000130 }
131
132 // popStackAfter - Pop the current value off of the top of the FP stack
133 // after the specified instruction.
134 void popStackAfter(MachineBasicBlock::iterator &I);
135
Chris Lattner0526f012004-04-01 04:06:09 +0000136 // freeStackSlotAfter - Free the specified register from the register stack,
137 // so that it is no longer in a register. If the register is currently at
138 // the top of the stack, we just pop the current instruction, otherwise we
139 // store the current top-of-stack into the specified slot, then pop the top
140 // of stack.
141 void freeStackSlotAfter(MachineBasicBlock::iterator &I, unsigned Reg);
142
Chris Lattnera960d952003-01-13 01:01:59 +0000143 bool processBasicBlock(MachineFunction &MF, MachineBasicBlock &MBB);
144
145 void handleZeroArgFP(MachineBasicBlock::iterator &I);
146 void handleOneArgFP(MachineBasicBlock::iterator &I);
Chris Lattner4a06f352004-02-02 19:23:15 +0000147 void handleOneArgFPRW(MachineBasicBlock::iterator &I);
Chris Lattnera960d952003-01-13 01:01:59 +0000148 void handleTwoArgFP(MachineBasicBlock::iterator &I);
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000149 void handleCompareFP(MachineBasicBlock::iterator &I);
Chris Lattnerc1bab322004-03-31 22:02:36 +0000150 void handleCondMovFP(MachineBasicBlock::iterator &I);
Chris Lattnera960d952003-01-13 01:01:59 +0000151 void handleSpecialFP(MachineBasicBlock::iterator &I);
152 };
153}
154
Chris Lattnerf2e49d42003-12-20 09:58:55 +0000155FunctionPass *llvm::createX86FloatingPointStackifierPass() { return new FPS(); }
Chris Lattnera960d952003-01-13 01:01:59 +0000156
157/// runOnMachineFunction - Loop over all of the basic blocks, transforming FP
158/// register references into FP stack references.
159///
160bool FPS::runOnMachineFunction(MachineFunction &MF) {
Chris Lattner42e25b32005-01-23 23:13:59 +0000161 // We only need to run this pass if there are any FP registers used in this
162 // function. If it is all integer, there is nothing for us to do!
163 const bool *PhysRegsUsed = MF.getUsedPhysregs();
164 bool FPIsUsed = false;
165
166 assert(X86::FP6 == X86::FP0+6 && "Register enums aren't sorted right!");
167 for (unsigned i = 0; i <= 6; ++i)
168 if (PhysRegsUsed[X86::FP0+i]) {
169 FPIsUsed = true;
170 break;
171 }
172
173 // Early exit.
174 if (!FPIsUsed) return false;
175
Chris Lattnera960d952003-01-13 01:01:59 +0000176 LV = &getAnalysis<LiveVariables>();
177 StackTop = 0;
178
Chris Lattner847df252004-01-30 22:25:18 +0000179 // Process the function in depth first order so that we process at least one
180 // of the predecessors for every reachable block in the function.
Chris Lattner22686842004-05-01 21:27:53 +0000181 std::set<MachineBasicBlock*> Processed;
182 MachineBasicBlock *Entry = MF.begin();
Chris Lattner847df252004-01-30 22:25:18 +0000183
184 bool Changed = false;
Chris Lattner22686842004-05-01 21:27:53 +0000185 for (df_ext_iterator<MachineBasicBlock*, std::set<MachineBasicBlock*> >
Chris Lattner847df252004-01-30 22:25:18 +0000186 I = df_ext_begin(Entry, Processed), E = df_ext_end(Entry, Processed);
187 I != E; ++I)
Chris Lattner22686842004-05-01 21:27:53 +0000188 Changed |= processBasicBlock(MF, **I);
Chris Lattner847df252004-01-30 22:25:18 +0000189
Chris Lattnera960d952003-01-13 01:01:59 +0000190 return Changed;
191}
192
193/// processBasicBlock - Loop over all of the instructions in the basic block,
194/// transforming FP instructions into their stack form.
195///
196bool FPS::processBasicBlock(MachineFunction &MF, MachineBasicBlock &BB) {
Chris Lattnerd029cd22004-06-02 05:55:25 +0000197 const TargetInstrInfo &TII = *MF.getTarget().getInstrInfo();
Chris Lattnera960d952003-01-13 01:01:59 +0000198 bool Changed = false;
199 MBB = &BB;
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000200
Chris Lattnera960d952003-01-13 01:01:59 +0000201 for (MachineBasicBlock::iterator I = BB.begin(); I != BB.end(); ++I) {
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000202 MachineInstr *MI = I;
Chris Lattnera960d952003-01-13 01:01:59 +0000203 unsigned Flags = TII.get(MI->getOpcode()).TSFlags;
Chris Lattner847df252004-01-30 22:25:18 +0000204 if ((Flags & X86II::FPTypeMask) == X86II::NotFP)
205 continue; // Efficiently ignore non-fp insts!
Chris Lattnera960d952003-01-13 01:01:59 +0000206
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000207 MachineInstr *PrevMI = 0;
Alkis Evlogimenosf81af212004-02-14 01:18:34 +0000208 if (I != BB.begin())
209 PrevMI = prior(I);
Chris Lattnera960d952003-01-13 01:01:59 +0000210
211 ++NumFP; // Keep track of # of pseudo instrs
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000212 DEBUG(std::cerr << "\nFPInst:\t"; MI->print(std::cerr, &(MF.getTarget())));
Chris Lattnera960d952003-01-13 01:01:59 +0000213
214 // Get dead variables list now because the MI pointer may be deleted as part
215 // of processing!
Chris Lattnerd1775792005-08-23 23:41:14 +0000216 LiveVariables::killed_iterator IB, IE;
217 tie(IB, IE) = LV->dead_range(MI);
Chris Lattnera960d952003-01-13 01:01:59 +0000218
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000219 DEBUG(
220 const MRegisterInfo *MRI = MF.getTarget().getRegisterInfo();
221 LiveVariables::killed_iterator I = LV->killed_begin(MI);
222 LiveVariables::killed_iterator E = LV->killed_end(MI);
223 if (I != E) {
224 std::cerr << "Killed Operands:";
225 for (; I != E; ++I)
Chris Lattnerd1775792005-08-23 23:41:14 +0000226 std::cerr << " %" << MRI->getName(*I);
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000227 std::cerr << "\n";
228 }
229 );
Chris Lattnera960d952003-01-13 01:01:59 +0000230
231 switch (Flags & X86II::FPTypeMask) {
Chris Lattner4a06f352004-02-02 19:23:15 +0000232 case X86II::ZeroArgFP: handleZeroArgFP(I); break;
Chris Lattnerc1bab322004-03-31 22:02:36 +0000233 case X86II::OneArgFP: handleOneArgFP(I); break; // fstp ST(0)
Chris Lattner4a06f352004-02-02 19:23:15 +0000234 case X86II::OneArgFPRW: handleOneArgFPRW(I); break; // ST(0) = fsqrt(ST(0))
Chris Lattnerab8decc2004-06-11 04:41:24 +0000235 case X86II::TwoArgFP: handleTwoArgFP(I); break;
236 case X86II::CompareFP: handleCompareFP(I); break;
Chris Lattnerc1bab322004-03-31 22:02:36 +0000237 case X86II::CondMovFP: handleCondMovFP(I); break;
Chris Lattner4a06f352004-02-02 19:23:15 +0000238 case X86II::SpecialFP: handleSpecialFP(I); break;
Chris Lattnera960d952003-01-13 01:01:59 +0000239 default: assert(0 && "Unknown FP Type!");
240 }
241
242 // Check to see if any of the values defined by this instruction are dead
243 // after definition. If so, pop them.
244 for (; IB != IE; ++IB) {
Chris Lattnerd1775792005-08-23 23:41:14 +0000245 unsigned Reg = *IB;
Chris Lattnera960d952003-01-13 01:01:59 +0000246 if (Reg >= X86::FP0 && Reg <= X86::FP6) {
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000247 DEBUG(std::cerr << "Register FP#" << Reg-X86::FP0 << " is dead!\n");
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000248 freeStackSlotAfter(I, Reg-X86::FP0);
Chris Lattnera960d952003-01-13 01:01:59 +0000249 }
250 }
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000251
Chris Lattnera960d952003-01-13 01:01:59 +0000252 // Print out all of the instructions expanded to if -debug
Alkis Evlogimenosb929bca2004-02-15 00:46:41 +0000253 DEBUG(
254 MachineBasicBlock::iterator PrevI(PrevMI);
255 if (I == PrevI) {
Chris Lattner0526f012004-04-01 04:06:09 +0000256 std::cerr << "Just deleted pseudo instruction\n";
Alkis Evlogimenosb929bca2004-02-15 00:46:41 +0000257 } else {
258 MachineBasicBlock::iterator Start = I;
259 // Rewind to first instruction newly inserted.
260 while (Start != BB.begin() && prior(Start) != PrevI) --Start;
261 std::cerr << "Inserted instructions:\n\t";
Tanya Lattnerb1407622004-06-25 00:13:11 +0000262 Start->print(std::cerr, &MF.getTarget());
Alkis Evlogimenosb929bca2004-02-15 00:46:41 +0000263 while (++Start != next(I));
264 }
265 dumpStack();
266 );
Chris Lattnera960d952003-01-13 01:01:59 +0000267
268 Changed = true;
269 }
270
271 assert(StackTop == 0 && "Stack not empty at end of basic block?");
272 return Changed;
273}
274
275//===----------------------------------------------------------------------===//
276// Efficient Lookup Table Support
277//===----------------------------------------------------------------------===//
278
Chris Lattnerf2e49d42003-12-20 09:58:55 +0000279namespace {
280 struct TableEntry {
281 unsigned from;
282 unsigned to;
283 bool operator<(const TableEntry &TE) const { return from < TE.from; }
284 bool operator<(unsigned V) const { return from < V; }
285 };
286}
Chris Lattnera960d952003-01-13 01:01:59 +0000287
288static bool TableIsSorted(const TableEntry *Table, unsigned NumEntries) {
289 for (unsigned i = 0; i != NumEntries-1; ++i)
290 if (!(Table[i] < Table[i+1])) return false;
291 return true;
292}
293
294static int Lookup(const TableEntry *Table, unsigned N, unsigned Opcode) {
295 const TableEntry *I = std::lower_bound(Table, Table+N, Opcode);
296 if (I != Table+N && I->from == Opcode)
297 return I->to;
298 return -1;
299}
300
301#define ARRAY_SIZE(TABLE) \
302 (sizeof(TABLE)/sizeof(TABLE[0]))
303
304#ifdef NDEBUG
305#define ASSERT_SORTED(TABLE)
306#else
307#define ASSERT_SORTED(TABLE) \
308 { static bool TABLE##Checked = false; \
309 if (!TABLE##Checked) \
310 assert(TableIsSorted(TABLE, ARRAY_SIZE(TABLE)) && \
311 "All lookup tables must be sorted for efficient access!"); \
312 }
313#endif
314
Chris Lattner58fe4592005-12-21 07:47:04 +0000315//===----------------------------------------------------------------------===//
316// Register File -> Register Stack Mapping Methods
317//===----------------------------------------------------------------------===//
318
319// OpcodeTable - Sorted map of register instructions to their stack version.
320// The first element is an register file pseudo instruction, the second is the
321// concrete X86 instruction which uses the register stack.
322//
323static const TableEntry OpcodeTable[] = {
Evan Chengf7100622006-01-10 22:22:02 +0000324 { X86::FpABS , X86::FABS },
325 { X86::FpADD32m , X86::FADD32m },
326 { X86::FpADD64m , X86::FADD64m },
327 { X86::FpCHS , X86::FCHS },
Evan Chengf7100622006-01-10 22:22:02 +0000328 { X86::FpCMOVB , X86::FCMOVB },
329 { X86::FpCMOVBE , X86::FCMOVBE },
330 { X86::FpCMOVE , X86::FCMOVE },
Evan Cheng86556a52006-01-21 02:55:41 +0000331 { X86::FpCMOVNB , X86::FCMOVNB },
332 { X86::FpCMOVNBE , X86::FCMOVNBE },
Evan Chengf7100622006-01-10 22:22:02 +0000333 { X86::FpCMOVNE , X86::FCMOVNE },
334 { X86::FpCMOVNP , X86::FCMOVNP },
335 { X86::FpCMOVP , X86::FCMOVP },
336 { X86::FpCOS , X86::FCOS },
337 { X86::FpDIV32m , X86::FDIV32m },
338 { X86::FpDIV64m , X86::FDIV64m },
339 { X86::FpDIVR32m , X86::FDIVR32m },
340 { X86::FpDIVR64m , X86::FDIVR64m },
341 { X86::FpIADD16m , X86::FIADD16m },
342 { X86::FpIADD32m , X86::FIADD32m },
343 { X86::FpIDIV16m , X86::FIDIV16m },
344 { X86::FpIDIV32m , X86::FIDIV32m },
345 { X86::FpIDIVR16m, X86::FIDIVR16m},
346 { X86::FpIDIVR32m, X86::FIDIVR32m},
347 { X86::FpILD16m , X86::FILD16m },
348 { X86::FpILD32m , X86::FILD32m },
349 { X86::FpILD64m , X86::FILD64m },
350 { X86::FpIMUL16m , X86::FIMUL16m },
351 { X86::FpIMUL32m , X86::FIMUL32m },
352 { X86::FpIST16m , X86::FIST16m },
353 { X86::FpIST32m , X86::FIST32m },
354 { X86::FpIST64m , X86::FISTP64m },
355 { X86::FpISUB16m , X86::FISUB16m },
356 { X86::FpISUB32m , X86::FISUB32m },
357 { X86::FpISUBR16m, X86::FISUBR16m},
358 { X86::FpISUBR32m, X86::FISUBR32m},
359 { X86::FpLD0 , X86::FLD0 },
360 { X86::FpLD1 , X86::FLD1 },
361 { X86::FpLD32m , X86::FLD32m },
362 { X86::FpLD64m , X86::FLD64m },
363 { X86::FpMUL32m , X86::FMUL32m },
364 { X86::FpMUL64m , X86::FMUL64m },
365 { X86::FpSIN , X86::FSIN },
366 { X86::FpSQRT , X86::FSQRT },
367 { X86::FpST32m , X86::FST32m },
368 { X86::FpST64m , X86::FST64m },
369 { X86::FpSUB32m , X86::FSUB32m },
370 { X86::FpSUB64m , X86::FSUB64m },
371 { X86::FpSUBR32m , X86::FSUBR32m },
372 { X86::FpSUBR64m , X86::FSUBR64m },
373 { X86::FpTST , X86::FTST },
374 { X86::FpUCOMIr , X86::FUCOMIr },
375 { X86::FpUCOMr , X86::FUCOMr },
Chris Lattner58fe4592005-12-21 07:47:04 +0000376};
377
378static unsigned getConcreteOpcode(unsigned Opcode) {
379 ASSERT_SORTED(OpcodeTable);
380 int Opc = Lookup(OpcodeTable, ARRAY_SIZE(OpcodeTable), Opcode);
381 assert(Opc != -1 && "FP Stack instruction not in OpcodeTable!");
382 return Opc;
383}
Chris Lattnera960d952003-01-13 01:01:59 +0000384
385//===----------------------------------------------------------------------===//
386// Helper Methods
387//===----------------------------------------------------------------------===//
388
389// PopTable - Sorted map of instructions to their popping version. The first
390// element is an instruction, the second is the version which pops.
391//
392static const TableEntry PopTable[] = {
Chris Lattner113455b2003-08-03 21:56:36 +0000393 { X86::FADDrST0 , X86::FADDPrST0 },
394
395 { X86::FDIVRrST0, X86::FDIVRPrST0 },
396 { X86::FDIVrST0 , X86::FDIVPrST0 },
397
Alkis Evlogimenos8295f202004-02-29 08:50:03 +0000398 { X86::FIST16m , X86::FISTP16m },
399 { X86::FIST32m , X86::FISTP32m },
Chris Lattnera960d952003-01-13 01:01:59 +0000400
Chris Lattnera960d952003-01-13 01:01:59 +0000401 { X86::FMULrST0 , X86::FMULPrST0 },
Chris Lattnera960d952003-01-13 01:01:59 +0000402
Alkis Evlogimenos8295f202004-02-29 08:50:03 +0000403 { X86::FST32m , X86::FSTP32m },
404 { X86::FST64m , X86::FSTP64m },
Chris Lattner113455b2003-08-03 21:56:36 +0000405 { X86::FSTrr , X86::FSTPrr },
406
407 { X86::FSUBRrST0, X86::FSUBRPrST0 },
408 { X86::FSUBrST0 , X86::FSUBPrST0 },
409
Chris Lattnerc040bca2004-04-12 01:39:15 +0000410 { X86::FUCOMIr , X86::FUCOMIPr },
411
Chris Lattnera960d952003-01-13 01:01:59 +0000412 { X86::FUCOMPr , X86::FUCOMPPr },
Chris Lattner113455b2003-08-03 21:56:36 +0000413 { X86::FUCOMr , X86::FUCOMPr },
Chris Lattnera960d952003-01-13 01:01:59 +0000414};
415
416/// popStackAfter - Pop the current value off of the top of the FP stack after
417/// the specified instruction. This attempts to be sneaky and combine the pop
418/// into the instruction itself if possible. The iterator is left pointing to
419/// the last instruction, be it a new pop instruction inserted, or the old
420/// instruction if it was modified in place.
421///
422void FPS::popStackAfter(MachineBasicBlock::iterator &I) {
423 ASSERT_SORTED(PopTable);
424 assert(StackTop > 0 && "Cannot pop empty stack!");
425 RegMap[Stack[--StackTop]] = ~0; // Update state
426
427 // Check to see if there is a popping version of this instruction...
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000428 int Opcode = Lookup(PopTable, ARRAY_SIZE(PopTable), I->getOpcode());
Chris Lattnera960d952003-01-13 01:01:59 +0000429 if (Opcode != -1) {
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000430 I->setOpcode(Opcode);
Chris Lattnera960d952003-01-13 01:01:59 +0000431 if (Opcode == X86::FUCOMPPr)
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000432 I->RemoveOperand(0);
Chris Lattnera960d952003-01-13 01:01:59 +0000433
434 } else { // Insert an explicit pop
Chris Lattner0526f012004-04-01 04:06:09 +0000435 I = BuildMI(*MBB, ++I, X86::FSTPrr, 1).addReg(X86::ST0);
Chris Lattnera960d952003-01-13 01:01:59 +0000436 }
437}
438
Chris Lattner0526f012004-04-01 04:06:09 +0000439/// freeStackSlotAfter - Free the specified register from the register stack, so
440/// that it is no longer in a register. If the register is currently at the top
441/// of the stack, we just pop the current instruction, otherwise we store the
442/// current top-of-stack into the specified slot, then pop the top of stack.
443void FPS::freeStackSlotAfter(MachineBasicBlock::iterator &I, unsigned FPRegNo) {
444 if (getStackEntry(0) == FPRegNo) { // already at the top of stack? easy.
445 popStackAfter(I);
446 return;
447 }
448
449 // Otherwise, store the top of stack into the dead slot, killing the operand
450 // without having to add in an explicit xchg then pop.
451 //
452 unsigned STReg = getSTReg(FPRegNo);
453 unsigned OldSlot = getSlot(FPRegNo);
454 unsigned TopReg = Stack[StackTop-1];
455 Stack[OldSlot] = TopReg;
456 RegMap[TopReg] = OldSlot;
457 RegMap[FPRegNo] = ~0;
458 Stack[--StackTop] = ~0;
459 I = BuildMI(*MBB, ++I, X86::FSTPrr, 1).addReg(STReg);
460}
461
462
Chris Lattnera960d952003-01-13 01:01:59 +0000463static unsigned getFPReg(const MachineOperand &MO) {
Chris Lattner6d215182004-02-10 20:31:28 +0000464 assert(MO.isRegister() && "Expected an FP register!");
Chris Lattnera960d952003-01-13 01:01:59 +0000465 unsigned Reg = MO.getReg();
466 assert(Reg >= X86::FP0 && Reg <= X86::FP6 && "Expected FP register!");
467 return Reg - X86::FP0;
468}
469
470
471//===----------------------------------------------------------------------===//
472// Instruction transformation implementation
473//===----------------------------------------------------------------------===//
474
475/// handleZeroArgFP - ST(0) = fld0 ST(0) = flds <mem>
Chris Lattner4a06f352004-02-02 19:23:15 +0000476///
Chris Lattnera960d952003-01-13 01:01:59 +0000477void FPS::handleZeroArgFP(MachineBasicBlock::iterator &I) {
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000478 MachineInstr *MI = I;
Chris Lattnera960d952003-01-13 01:01:59 +0000479 unsigned DestReg = getFPReg(MI->getOperand(0));
Chris Lattnera960d952003-01-13 01:01:59 +0000480
Chris Lattner58fe4592005-12-21 07:47:04 +0000481 // Change from the pseudo instruction to the concrete instruction.
482 MI->RemoveOperand(0); // Remove the explicit ST(0) operand
483 MI->setOpcode(getConcreteOpcode(MI->getOpcode()));
484
485 // Result gets pushed on the stack.
Chris Lattnera960d952003-01-13 01:01:59 +0000486 pushReg(DestReg);
487}
488
Chris Lattner4a06f352004-02-02 19:23:15 +0000489/// handleOneArgFP - fst <mem>, ST(0)
490///
Chris Lattnera960d952003-01-13 01:01:59 +0000491void FPS::handleOneArgFP(MachineBasicBlock::iterator &I) {
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000492 MachineInstr *MI = I;
Chris Lattnerb97046a2004-02-03 07:27:34 +0000493 assert((MI->getNumOperands() == 5 || MI->getNumOperands() == 1) &&
494 "Can only handle fst* & ftst instructions!");
Chris Lattnera960d952003-01-13 01:01:59 +0000495
Chris Lattner4a06f352004-02-02 19:23:15 +0000496 // Is this the last use of the source register?
Chris Lattnerb97046a2004-02-03 07:27:34 +0000497 unsigned Reg = getFPReg(MI->getOperand(MI->getNumOperands()-1));
Chris Lattner76eb08b2005-08-23 22:49:55 +0000498 bool KillsSrc = LV->KillsRegister(MI, X86::FP0+Reg);
Chris Lattnera960d952003-01-13 01:01:59 +0000499
Chris Lattner58fe4592005-12-21 07:47:04 +0000500 // FISTP64r is strange because there isn't a non-popping versions.
Chris Lattnera960d952003-01-13 01:01:59 +0000501 // If we have one _and_ we don't want to pop the operand, duplicate the value
502 // on the stack instead of moving it. This ensure that popping the value is
503 // always ok.
504 //
Chris Lattner58fe4592005-12-21 07:47:04 +0000505 if (MI->getOpcode() == X86::FpIST64m && !KillsSrc) {
Chris Lattnera960d952003-01-13 01:01:59 +0000506 duplicateToTop(Reg, 7 /*temp register*/, I);
507 } else {
508 moveToTop(Reg, I); // Move to the top of the stack...
509 }
Chris Lattner58fe4592005-12-21 07:47:04 +0000510
511 // Convert from the pseudo instruction to the concrete instruction.
Chris Lattnerb97046a2004-02-03 07:27:34 +0000512 MI->RemoveOperand(MI->getNumOperands()-1); // Remove explicit ST(0) operand
Chris Lattner58fe4592005-12-21 07:47:04 +0000513 MI->setOpcode(getConcreteOpcode(MI->getOpcode()));
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000514
Chris Lattner58fe4592005-12-21 07:47:04 +0000515 if (MI->getOpcode() == X86::FISTP64m) {
Chris Lattnera960d952003-01-13 01:01:59 +0000516 assert(StackTop > 0 && "Stack empty??");
517 --StackTop;
518 } else if (KillsSrc) { // Last use of operand?
519 popStackAfter(I);
520 }
521}
522
Chris Lattner4a06f352004-02-02 19:23:15 +0000523
Chris Lattner4cf15e72004-04-11 20:21:06 +0000524/// handleOneArgFPRW: Handle instructions that read from the top of stack and
525/// replace the value with a newly computed value. These instructions may have
526/// non-fp operands after their FP operands.
527///
528/// Examples:
529/// R1 = fchs R2
530/// R1 = fadd R2, [mem]
Chris Lattner4a06f352004-02-02 19:23:15 +0000531///
532void FPS::handleOneArgFPRW(MachineBasicBlock::iterator &I) {
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000533 MachineInstr *MI = I;
Chris Lattner4cf15e72004-04-11 20:21:06 +0000534 assert(MI->getNumOperands() >= 2 && "FPRW instructions must have 2 ops!!");
Chris Lattner4a06f352004-02-02 19:23:15 +0000535
536 // Is this the last use of the source register?
537 unsigned Reg = getFPReg(MI->getOperand(1));
Chris Lattner76eb08b2005-08-23 22:49:55 +0000538 bool KillsSrc = LV->KillsRegister(MI, X86::FP0+Reg);
Chris Lattner4a06f352004-02-02 19:23:15 +0000539
540 if (KillsSrc) {
541 // If this is the last use of the source register, just make sure it's on
542 // the top of the stack.
543 moveToTop(Reg, I);
544 assert(StackTop > 0 && "Stack cannot be empty!");
545 --StackTop;
546 pushReg(getFPReg(MI->getOperand(0)));
547 } else {
548 // If this is not the last use of the source register, _copy_ it to the top
549 // of the stack.
550 duplicateToTop(Reg, getFPReg(MI->getOperand(0)), I);
551 }
552
Chris Lattner58fe4592005-12-21 07:47:04 +0000553 // Change from the pseudo instruction to the concrete instruction.
Chris Lattner4a06f352004-02-02 19:23:15 +0000554 MI->RemoveOperand(1); // Drop the source operand.
555 MI->RemoveOperand(0); // Drop the destination operand.
Chris Lattner58fe4592005-12-21 07:47:04 +0000556 MI->setOpcode(getConcreteOpcode(MI->getOpcode()));
Chris Lattner4a06f352004-02-02 19:23:15 +0000557}
558
559
Chris Lattnera960d952003-01-13 01:01:59 +0000560//===----------------------------------------------------------------------===//
561// Define tables of various ways to map pseudo instructions
562//
563
564// ForwardST0Table - Map: A = B op C into: ST(0) = ST(0) op ST(i)
565static const TableEntry ForwardST0Table[] = {
Chris Lattnerc040bca2004-04-12 01:39:15 +0000566 { X86::FpADD , X86::FADDST0r },
567 { X86::FpDIV , X86::FDIVST0r },
568 { X86::FpMUL , X86::FMULST0r },
569 { X86::FpSUB , X86::FSUBST0r },
Chris Lattnera960d952003-01-13 01:01:59 +0000570};
571
572// ReverseST0Table - Map: A = B op C into: ST(0) = ST(i) op ST(0)
573static const TableEntry ReverseST0Table[] = {
Chris Lattnerc040bca2004-04-12 01:39:15 +0000574 { X86::FpADD , X86::FADDST0r }, // commutative
575 { X86::FpDIV , X86::FDIVRST0r },
576 { X86::FpMUL , X86::FMULST0r }, // commutative
577 { X86::FpSUB , X86::FSUBRST0r },
Chris Lattnera960d952003-01-13 01:01:59 +0000578};
579
580// ForwardSTiTable - Map: A = B op C into: ST(i) = ST(0) op ST(i)
581static const TableEntry ForwardSTiTable[] = {
Chris Lattnerc040bca2004-04-12 01:39:15 +0000582 { X86::FpADD , X86::FADDrST0 }, // commutative
583 { X86::FpDIV , X86::FDIVRrST0 },
584 { X86::FpMUL , X86::FMULrST0 }, // commutative
585 { X86::FpSUB , X86::FSUBRrST0 },
Chris Lattnera960d952003-01-13 01:01:59 +0000586};
587
588// ReverseSTiTable - Map: A = B op C into: ST(i) = ST(i) op ST(0)
589static const TableEntry ReverseSTiTable[] = {
Chris Lattnerc040bca2004-04-12 01:39:15 +0000590 { X86::FpADD , X86::FADDrST0 },
591 { X86::FpDIV , X86::FDIVrST0 },
592 { X86::FpMUL , X86::FMULrST0 },
593 { X86::FpSUB , X86::FSUBrST0 },
Chris Lattnera960d952003-01-13 01:01:59 +0000594};
595
596
597/// handleTwoArgFP - Handle instructions like FADD and friends which are virtual
598/// instructions which need to be simplified and possibly transformed.
599///
600/// Result: ST(0) = fsub ST(0), ST(i)
601/// ST(i) = fsub ST(0), ST(i)
602/// ST(0) = fsubr ST(0), ST(i)
603/// ST(i) = fsubr ST(0), ST(i)
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000604///
Chris Lattnera960d952003-01-13 01:01:59 +0000605void FPS::handleTwoArgFP(MachineBasicBlock::iterator &I) {
606 ASSERT_SORTED(ForwardST0Table); ASSERT_SORTED(ReverseST0Table);
607 ASSERT_SORTED(ForwardSTiTable); ASSERT_SORTED(ReverseSTiTable);
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000608 MachineInstr *MI = I;
Chris Lattnera960d952003-01-13 01:01:59 +0000609
610 unsigned NumOperands = MI->getNumOperands();
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000611 assert(NumOperands == 3 && "Illegal TwoArgFP instruction!");
Chris Lattnera960d952003-01-13 01:01:59 +0000612 unsigned Dest = getFPReg(MI->getOperand(0));
613 unsigned Op0 = getFPReg(MI->getOperand(NumOperands-2));
614 unsigned Op1 = getFPReg(MI->getOperand(NumOperands-1));
Chris Lattner76eb08b2005-08-23 22:49:55 +0000615 bool KillsOp0 = LV->KillsRegister(MI, X86::FP0+Op0);
616 bool KillsOp1 = LV->KillsRegister(MI, X86::FP0+Op1);
Chris Lattnera960d952003-01-13 01:01:59 +0000617
Chris Lattnera960d952003-01-13 01:01:59 +0000618 unsigned TOS = getStackEntry(0);
619
620 // One of our operands must be on the top of the stack. If neither is yet, we
621 // need to move one.
622 if (Op0 != TOS && Op1 != TOS) { // No operand at TOS?
623 // We can choose to move either operand to the top of the stack. If one of
624 // the operands is killed by this instruction, we want that one so that we
625 // can update right on top of the old version.
626 if (KillsOp0) {
627 moveToTop(Op0, I); // Move dead operand to TOS.
628 TOS = Op0;
629 } else if (KillsOp1) {
630 moveToTop(Op1, I);
631 TOS = Op1;
632 } else {
633 // All of the operands are live after this instruction executes, so we
634 // cannot update on top of any operand. Because of this, we must
635 // duplicate one of the stack elements to the top. It doesn't matter
636 // which one we pick.
637 //
638 duplicateToTop(Op0, Dest, I);
639 Op0 = TOS = Dest;
640 KillsOp0 = true;
641 }
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000642 } else if (!KillsOp0 && !KillsOp1) {
Chris Lattnera960d952003-01-13 01:01:59 +0000643 // If we DO have one of our operands at the top of the stack, but we don't
644 // have a dead operand, we must duplicate one of the operands to a new slot
645 // on the stack.
646 duplicateToTop(Op0, Dest, I);
647 Op0 = TOS = Dest;
648 KillsOp0 = true;
649 }
650
651 // Now we know that one of our operands is on the top of the stack, and at
652 // least one of our operands is killed by this instruction.
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000653 assert((TOS == Op0 || TOS == Op1) && (KillsOp0 || KillsOp1) &&
654 "Stack conditions not set up right!");
Chris Lattnera960d952003-01-13 01:01:59 +0000655
656 // We decide which form to use based on what is on the top of the stack, and
657 // which operand is killed by this instruction.
658 const TableEntry *InstTable;
659 bool isForward = TOS == Op0;
660 bool updateST0 = (TOS == Op0 && !KillsOp1) || (TOS == Op1 && !KillsOp0);
661 if (updateST0) {
662 if (isForward)
663 InstTable = ForwardST0Table;
664 else
665 InstTable = ReverseST0Table;
666 } else {
667 if (isForward)
668 InstTable = ForwardSTiTable;
669 else
670 InstTable = ReverseSTiTable;
671 }
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000672
Chris Lattnera960d952003-01-13 01:01:59 +0000673 int Opcode = Lookup(InstTable, ARRAY_SIZE(ForwardST0Table), MI->getOpcode());
674 assert(Opcode != -1 && "Unknown TwoArgFP pseudo instruction!");
675
676 // NotTOS - The register which is not on the top of stack...
677 unsigned NotTOS = (TOS == Op0) ? Op1 : Op0;
678
679 // Replace the old instruction with a new instruction
Chris Lattnerc1bab322004-03-31 22:02:36 +0000680 MBB->remove(I++);
Chris Lattner0526f012004-04-01 04:06:09 +0000681 I = BuildMI(*MBB, I, Opcode, 1).addReg(getSTReg(NotTOS));
Chris Lattnera960d952003-01-13 01:01:59 +0000682
683 // If both operands are killed, pop one off of the stack in addition to
684 // overwriting the other one.
685 if (KillsOp0 && KillsOp1 && Op0 != Op1) {
686 assert(!updateST0 && "Should have updated other operand!");
687 popStackAfter(I); // Pop the top of stack
688 }
689
Chris Lattnera960d952003-01-13 01:01:59 +0000690 // Update stack information so that we know the destination register is now on
691 // the stack.
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000692 unsigned UpdatedSlot = getSlot(updateST0 ? TOS : NotTOS);
693 assert(UpdatedSlot < StackTop && Dest < 7);
694 Stack[UpdatedSlot] = Dest;
695 RegMap[Dest] = UpdatedSlot;
696 delete MI; // Remove the old instruction
697}
698
Chris Lattner0ca2c8e2004-06-11 04:49:02 +0000699/// handleCompareFP - Handle FUCOM and FUCOMI instructions, which have two FP
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000700/// register arguments and no explicit destinations.
Misha Brukman0e0a7a452005-04-21 23:38:14 +0000701///
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000702void FPS::handleCompareFP(MachineBasicBlock::iterator &I) {
703 ASSERT_SORTED(ForwardST0Table); ASSERT_SORTED(ReverseST0Table);
704 ASSERT_SORTED(ForwardSTiTable); ASSERT_SORTED(ReverseSTiTable);
705 MachineInstr *MI = I;
706
707 unsigned NumOperands = MI->getNumOperands();
Chris Lattner0ca2c8e2004-06-11 04:49:02 +0000708 assert(NumOperands == 2 && "Illegal FUCOM* instruction!");
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000709 unsigned Op0 = getFPReg(MI->getOperand(NumOperands-2));
710 unsigned Op1 = getFPReg(MI->getOperand(NumOperands-1));
Chris Lattner76eb08b2005-08-23 22:49:55 +0000711 bool KillsOp0 = LV->KillsRegister(MI, X86::FP0+Op0);
712 bool KillsOp1 = LV->KillsRegister(MI, X86::FP0+Op1);
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000713
714 // Make sure the first operand is on the top of stack, the other one can be
715 // anywhere.
716 moveToTop(Op0, I);
717
Chris Lattner58fe4592005-12-21 07:47:04 +0000718 // Change from the pseudo instruction to the concrete instruction.
Chris Lattner57790422004-06-11 05:22:44 +0000719 MI->getOperand(0).setReg(getSTReg(Op1));
720 MI->RemoveOperand(1);
Chris Lattner58fe4592005-12-21 07:47:04 +0000721 MI->setOpcode(getConcreteOpcode(MI->getOpcode()));
Chris Lattner57790422004-06-11 05:22:44 +0000722
Chris Lattnerd62d5d72004-06-11 04:25:06 +0000723 // If any of the operands are killed by this instruction, free them.
724 if (KillsOp0) freeStackSlotAfter(I, Op0);
725 if (KillsOp1 && Op0 != Op1) freeStackSlotAfter(I, Op1);
Chris Lattnera960d952003-01-13 01:01:59 +0000726}
727
Chris Lattnerc1bab322004-03-31 22:02:36 +0000728/// handleCondMovFP - Handle two address conditional move instructions. These
729/// instructions move a st(i) register to st(0) iff a condition is true. These
730/// instructions require that the first operand is at the top of the stack, but
731/// otherwise don't modify the stack at all.
732void FPS::handleCondMovFP(MachineBasicBlock::iterator &I) {
733 MachineInstr *MI = I;
734
735 unsigned Op0 = getFPReg(MI->getOperand(0));
736 unsigned Op1 = getFPReg(MI->getOperand(1));
737
738 // The first operand *must* be on the top of the stack.
739 moveToTop(Op0, I);
740
741 // Change the second operand to the stack register that the operand is in.
Chris Lattner58fe4592005-12-21 07:47:04 +0000742 // Change from the pseudo instruction to the concrete instruction.
Chris Lattnerc1bab322004-03-31 22:02:36 +0000743 MI->RemoveOperand(0);
744 MI->getOperand(0).setReg(getSTReg(Op1));
Chris Lattner58fe4592005-12-21 07:47:04 +0000745 MI->setOpcode(getConcreteOpcode(MI->getOpcode()));
746
747
Chris Lattnerc1bab322004-03-31 22:02:36 +0000748 // If we kill the second operand, make sure to pop it from the stack.
Chris Lattner76eb08b2005-08-23 22:49:55 +0000749 if (Op0 != Op1 && LV->KillsRegister(MI, X86::FP0+Op1)) {
750 // Get this value off of the register stack.
751 freeStackSlotAfter(I, Op1);
752 }
Chris Lattnerc1bab322004-03-31 22:02:36 +0000753}
754
Chris Lattnera960d952003-01-13 01:01:59 +0000755
756/// handleSpecialFP - Handle special instructions which behave unlike other
Misha Brukmancf00c4a2003-10-10 17:57:28 +0000757/// floating point instructions. This is primarily intended for use by pseudo
Chris Lattnera960d952003-01-13 01:01:59 +0000758/// instructions.
759///
760void FPS::handleSpecialFP(MachineBasicBlock::iterator &I) {
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000761 MachineInstr *MI = I;
Chris Lattnera960d952003-01-13 01:01:59 +0000762 switch (MI->getOpcode()) {
763 default: assert(0 && "Unknown SpecialFP instruction!");
764 case X86::FpGETRESULT: // Appears immediately after a call returning FP type!
765 assert(StackTop == 0 && "Stack should be empty after a call!");
766 pushReg(getFPReg(MI->getOperand(0)));
767 break;
768 case X86::FpSETRESULT:
769 assert(StackTop == 1 && "Stack should have one element on it to return!");
770 --StackTop; // "Forget" we have something on the top of stack!
771 break;
772 case X86::FpMOV: {
773 unsigned SrcReg = getFPReg(MI->getOperand(1));
774 unsigned DestReg = getFPReg(MI->getOperand(0));
Chris Lattnera960d952003-01-13 01:01:59 +0000775
Chris Lattner76eb08b2005-08-23 22:49:55 +0000776 if (LV->KillsRegister(MI, X86::FP0+SrcReg)) {
Chris Lattnera960d952003-01-13 01:01:59 +0000777 // If the input operand is killed, we can just change the owner of the
778 // incoming stack slot into the result.
779 unsigned Slot = getSlot(SrcReg);
780 assert(Slot < 7 && DestReg < 7 && "FpMOV operands invalid!");
781 Stack[Slot] = DestReg;
782 RegMap[DestReg] = Slot;
783
784 } else {
785 // For FMOV we just duplicate the specified value to a new stack slot.
786 // This could be made better, but would require substantial changes.
787 duplicateToTop(SrcReg, DestReg, I);
788 }
789 break;
790 }
791 }
792
Alkis Evlogimenosc0b9dc52004-02-12 02:27:10 +0000793 I = MBB->erase(I); // Remove the pseudo instruction
794 --I;
Chris Lattnera960d952003-01-13 01:01:59 +0000795}