blob: 19d41f4dc2e9b25d2504eaf773ecb2ee69f4a31c [file] [log] [blame]
Bill Wendling6cdb1ab2010-08-09 23:59:04 +00001//===-- PeepholeOptimizer.cpp - Peephole Optimizations --------------------===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// Perform peephole optimizations on the machine code:
11//
12// - Optimize Extensions
13//
14// Optimization of sign / zero extension instructions. It may be extended to
15// handle other instructions with similar properties.
16//
17// On some targets, some instructions, e.g. X86 sign / zero extension, may
18// leave the source value in the lower part of the result. This optimization
19// will replace some uses of the pre-extension value with uses of the
20// sub-register of the results.
21//
22// - Optimize Comparisons
23//
24// Optimization of comparison instructions. For instance, in this code:
25//
26// sub r1, 1
27// cmp r1, 0
28// bz L1
29//
30// If the "sub" instruction all ready sets (or could be modified to set) the
31// same flag that the "cmp" instruction sets and that "bz" uses, then we can
32// eliminate the "cmp" instruction.
Evan Chengd158fba2011-03-15 05:13:13 +000033//
34// - Optimize Bitcast pairs:
35//
36// v1 = bitcast v0
37// v2 = bitcast v1
38// = v2
39// =>
40// v1 = bitcast v0
41// = v0
Andrew Trick1df91b02012-02-08 21:22:43 +000042//
Bill Wendling6cdb1ab2010-08-09 23:59:04 +000043//===----------------------------------------------------------------------===//
44
45#define DEBUG_TYPE "peephole-opt"
46#include "llvm/CodeGen/Passes.h"
47#include "llvm/CodeGen/MachineDominators.h"
48#include "llvm/CodeGen/MachineInstrBuilder.h"
49#include "llvm/CodeGen/MachineRegisterInfo.h"
50#include "llvm/Target/TargetInstrInfo.h"
51#include "llvm/Target/TargetRegisterInfo.h"
52#include "llvm/Support/CommandLine.h"
Evan Chengc4af4632010-11-17 20:13:28 +000053#include "llvm/ADT/DenseMap.h"
Bill Wendling6cdb1ab2010-08-09 23:59:04 +000054#include "llvm/ADT/SmallPtrSet.h"
Evan Chengc4af4632010-11-17 20:13:28 +000055#include "llvm/ADT/SmallSet.h"
Bill Wendling6cdb1ab2010-08-09 23:59:04 +000056#include "llvm/ADT/Statistic.h"
57using namespace llvm;
58
59// Optimize Extensions
60static cl::opt<bool>
61Aggressive("aggressive-ext-opt", cl::Hidden,
62 cl::desc("Aggressive extension optimization"));
63
Bill Wendling40a5eb12010-11-01 20:41:43 +000064static cl::opt<bool>
65DisablePeephole("disable-peephole", cl::Hidden, cl::init(false),
66 cl::desc("Disable the peephole optimizer"));
67
Bill Wendling69c5eb52010-08-27 20:39:09 +000068STATISTIC(NumReuse, "Number of extension results reused");
Evan Chengd158fba2011-03-15 05:13:13 +000069STATISTIC(NumBitcasts, "Number of bitcasts eliminated");
70STATISTIC(NumCmps, "Number of compares eliminated");
Evan Chengc4af4632010-11-17 20:13:28 +000071STATISTIC(NumImmFold, "Number of move immediate foled");
Bill Wendling6cdb1ab2010-08-09 23:59:04 +000072
73namespace {
74 class PeepholeOptimizer : public MachineFunctionPass {
75 const TargetMachine *TM;
76 const TargetInstrInfo *TII;
77 MachineRegisterInfo *MRI;
78 MachineDominatorTree *DT; // Machine dominator tree
79
80 public:
81 static char ID; // Pass identification
Owen Anderson081c34b2010-10-19 17:21:58 +000082 PeepholeOptimizer() : MachineFunctionPass(ID) {
83 initializePeepholeOptimizerPass(*PassRegistry::getPassRegistry());
84 }
Bill Wendling6cdb1ab2010-08-09 23:59:04 +000085
86 virtual bool runOnMachineFunction(MachineFunction &MF);
87
88 virtual void getAnalysisUsage(AnalysisUsage &AU) const {
89 AU.setPreservesCFG();
90 MachineFunctionPass::getAnalysisUsage(AU);
91 if (Aggressive) {
92 AU.addRequired<MachineDominatorTree>();
93 AU.addPreserved<MachineDominatorTree>();
94 }
95 }
96
97 private:
Evan Chengd158fba2011-03-15 05:13:13 +000098 bool OptimizeBitcastInstr(MachineInstr *MI, MachineBasicBlock *MBB);
Evan Chengeb96a2f2010-11-15 21:20:45 +000099 bool OptimizeCmpInstr(MachineInstr *MI, MachineBasicBlock *MBB);
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000100 bool OptimizeExtInstr(MachineInstr *MI, MachineBasicBlock *MBB,
101 SmallPtrSet<MachineInstr*, 8> &LocalMIs);
Evan Chengc4af4632010-11-17 20:13:28 +0000102 bool isMoveImmediate(MachineInstr *MI,
103 SmallSet<unsigned, 4> &ImmDefRegs,
104 DenseMap<unsigned, MachineInstr*> &ImmDefMIs);
105 bool FoldImmediate(MachineInstr *MI, MachineBasicBlock *MBB,
106 SmallSet<unsigned, 4> &ImmDefRegs,
107 DenseMap<unsigned, MachineInstr*> &ImmDefMIs);
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000108 };
109}
110
111char PeepholeOptimizer::ID = 0;
Andrew Trick1dd8c852012-02-08 21:23:13 +0000112char &llvm::PeepholeOptimizerID = PeepholeOptimizer::ID;
Owen Anderson2ab36d32010-10-12 19:48:12 +0000113INITIALIZE_PASS_BEGIN(PeepholeOptimizer, "peephole-opts",
114 "Peephole Optimizations", false, false)
115INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree)
116INITIALIZE_PASS_END(PeepholeOptimizer, "peephole-opts",
Owen Andersonce665bd2010-10-07 22:25:06 +0000117 "Peephole Optimizations", false, false)
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000118
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000119/// OptimizeExtInstr - If instruction is a copy-like instruction, i.e. it reads
120/// a single register and writes a single register and it does not modify the
121/// source, and if the source value is preserved as a sub-register of the
122/// result, then replace all reachable uses of the source with the subreg of the
123/// result.
Andrew Trick1df91b02012-02-08 21:22:43 +0000124///
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000125/// Do not generate an EXTRACT that is used only in a debug use, as this changes
126/// the code. Since this code does not currently share EXTRACTs, just ignore all
127/// debug uses.
128bool PeepholeOptimizer::
129OptimizeExtInstr(MachineInstr *MI, MachineBasicBlock *MBB,
130 SmallPtrSet<MachineInstr*, 8> &LocalMIs) {
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000131 unsigned SrcReg, DstReg, SubIdx;
132 if (!TII->isCoalescableExtInstr(*MI, SrcReg, DstReg, SubIdx))
133 return false;
Andrew Trick1df91b02012-02-08 21:22:43 +0000134
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000135 if (TargetRegisterInfo::isPhysicalRegister(DstReg) ||
136 TargetRegisterInfo::isPhysicalRegister(SrcReg))
137 return false;
138
139 MachineRegisterInfo::use_nodbg_iterator UI = MRI->use_nodbg_begin(SrcReg);
140 if (++UI == MRI->use_nodbg_end())
141 // No other uses.
142 return false;
143
144 // The source has other uses. See if we can replace the other uses with use of
145 // the result of the extension.
146 SmallPtrSet<MachineBasicBlock*, 4> ReachedBBs;
147 UI = MRI->use_nodbg_begin(DstReg);
148 for (MachineRegisterInfo::use_nodbg_iterator UE = MRI->use_nodbg_end();
149 UI != UE; ++UI)
150 ReachedBBs.insert(UI->getParent());
151
152 // Uses that are in the same BB of uses of the result of the instruction.
153 SmallVector<MachineOperand*, 8> Uses;
154
155 // Uses that the result of the instruction can reach.
156 SmallVector<MachineOperand*, 8> ExtendedUses;
157
158 bool ExtendLife = true;
159 UI = MRI->use_nodbg_begin(SrcReg);
160 for (MachineRegisterInfo::use_nodbg_iterator UE = MRI->use_nodbg_end();
161 UI != UE; ++UI) {
162 MachineOperand &UseMO = UI.getOperand();
163 MachineInstr *UseMI = &*UI;
164 if (UseMI == MI)
165 continue;
166
167 if (UseMI->isPHI()) {
168 ExtendLife = false;
169 continue;
170 }
171
172 // It's an error to translate this:
173 //
174 // %reg1025 = <sext> %reg1024
175 // ...
176 // %reg1026 = SUBREG_TO_REG 0, %reg1024, 4
177 //
178 // into this:
179 //
180 // %reg1025 = <sext> %reg1024
181 // ...
182 // %reg1027 = COPY %reg1025:4
183 // %reg1026 = SUBREG_TO_REG 0, %reg1027, 4
184 //
185 // The problem here is that SUBREG_TO_REG is there to assert that an
186 // implicit zext occurs. It doesn't insert a zext instruction. If we allow
187 // the COPY here, it will give us the value after the <sext>, not the
188 // original value of %reg1024 before <sext>.
189 if (UseMI->getOpcode() == TargetOpcode::SUBREG_TO_REG)
190 continue;
191
192 MachineBasicBlock *UseMBB = UseMI->getParent();
193 if (UseMBB == MBB) {
194 // Local uses that come after the extension.
195 if (!LocalMIs.count(UseMI))
196 Uses.push_back(&UseMO);
197 } else if (ReachedBBs.count(UseMBB)) {
198 // Non-local uses where the result of the extension is used. Always
199 // replace these unless it's a PHI.
200 Uses.push_back(&UseMO);
201 } else if (Aggressive && DT->dominates(MBB, UseMBB)) {
202 // We may want to extend the live range of the extension result in order
203 // to replace these uses.
204 ExtendedUses.push_back(&UseMO);
205 } else {
206 // Both will be live out of the def MBB anyway. Don't extend live range of
207 // the extension result.
208 ExtendLife = false;
209 break;
210 }
211 }
212
213 if (ExtendLife && !ExtendedUses.empty())
214 // Extend the liveness of the extension result.
215 std::copy(ExtendedUses.begin(), ExtendedUses.end(),
216 std::back_inserter(Uses));
217
218 // Now replace all uses.
219 bool Changed = false;
220 if (!Uses.empty()) {
221 SmallPtrSet<MachineBasicBlock*, 4> PHIBBs;
222
223 // Look for PHI uses of the extended result, we don't want to extend the
224 // liveness of a PHI input. It breaks all kinds of assumptions down
225 // stream. A PHI use is expected to be the kill of its source values.
226 UI = MRI->use_nodbg_begin(DstReg);
227 for (MachineRegisterInfo::use_nodbg_iterator
228 UE = MRI->use_nodbg_end(); UI != UE; ++UI)
229 if (UI->isPHI())
230 PHIBBs.insert(UI->getParent());
231
232 const TargetRegisterClass *RC = MRI->getRegClass(SrcReg);
233 for (unsigned i = 0, e = Uses.size(); i != e; ++i) {
234 MachineOperand *UseMO = Uses[i];
235 MachineInstr *UseMI = UseMO->getParent();
236 MachineBasicBlock *UseMBB = UseMI->getParent();
237 if (PHIBBs.count(UseMBB))
238 continue;
239
240 unsigned NewVR = MRI->createVirtualRegister(RC);
241 BuildMI(*UseMBB, UseMI, UseMI->getDebugLoc(),
242 TII->get(TargetOpcode::COPY), NewVR)
243 .addReg(DstReg, 0, SubIdx);
244
245 UseMO->setReg(NewVR);
246 ++NumReuse;
247 Changed = true;
248 }
249 }
250
251 return Changed;
252}
253
Evan Chengd158fba2011-03-15 05:13:13 +0000254/// OptimizeBitcastInstr - If the instruction is a bitcast instruction A that
255/// cannot be optimized away during isel (e.g. ARM::VMOVSR, which bitcast
256/// a value cross register classes), and the source is defined by another
257/// bitcast instruction B. And if the register class of source of B matches
258/// the register class of instruction A, then it is legal to replace all uses
259/// of the def of A with source of B. e.g.
260/// %vreg0<def> = VMOVSR %vreg1
261/// %vreg3<def> = VMOVRS %vreg0
262/// Replace all uses of vreg3 with vreg1.
263
264bool PeepholeOptimizer::OptimizeBitcastInstr(MachineInstr *MI,
265 MachineBasicBlock *MBB) {
266 unsigned NumDefs = MI->getDesc().getNumDefs();
267 unsigned NumSrcs = MI->getDesc().getNumOperands() - NumDefs;
268 if (NumDefs != 1)
269 return false;
270
271 unsigned Def = 0;
272 unsigned Src = 0;
273 for (unsigned i = 0, e = NumDefs + NumSrcs; i != e; ++i) {
274 const MachineOperand &MO = MI->getOperand(i);
275 if (!MO.isReg())
276 continue;
277 unsigned Reg = MO.getReg();
278 if (!Reg)
279 continue;
280 if (MO.isDef())
281 Def = Reg;
282 else if (Src)
283 // Multiple sources?
284 return false;
285 else
286 Src = Reg;
287 }
288
289 assert(Def && Src && "Malformed bitcast instruction!");
290
291 MachineInstr *DefMI = MRI->getVRegDef(Src);
Evan Cheng5a96b3d2011-12-07 07:15:52 +0000292 if (!DefMI || !DefMI->isBitcast())
Evan Chengd158fba2011-03-15 05:13:13 +0000293 return false;
294
Evan Chengd158fba2011-03-15 05:13:13 +0000295 unsigned SrcSrc = 0;
296 NumDefs = DefMI->getDesc().getNumDefs();
297 NumSrcs = DefMI->getDesc().getNumOperands() - NumDefs;
298 if (NumDefs != 1)
299 return false;
300 for (unsigned i = 0, e = NumDefs + NumSrcs; i != e; ++i) {
301 const MachineOperand &MO = DefMI->getOperand(i);
302 if (!MO.isReg() || MO.isDef())
303 continue;
304 unsigned Reg = MO.getReg();
305 if (!Reg)
306 continue;
Duncan Sands7becbc42011-07-26 15:05:06 +0000307 if (!MO.isDef()) {
308 if (SrcSrc)
309 // Multiple sources?
310 return false;
311 else
312 SrcSrc = Reg;
313 }
Evan Chengd158fba2011-03-15 05:13:13 +0000314 }
315
316 if (MRI->getRegClass(SrcSrc) != MRI->getRegClass(Def))
317 return false;
318
319 MRI->replaceRegWith(Def, SrcSrc);
320 MRI->clearKillFlags(SrcSrc);
321 MI->eraseFromParent();
322 ++NumBitcasts;
323 return true;
324}
325
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000326/// OptimizeCmpInstr - If the instruction is a compare and the previous
327/// instruction it's comparing against all ready sets (or could be modified to
328/// set) the same flag as the compare, then we can remove the comparison and use
329/// the flag from the previous instruction.
330bool PeepholeOptimizer::OptimizeCmpInstr(MachineInstr *MI,
Evan Chengd158fba2011-03-15 05:13:13 +0000331 MachineBasicBlock *MBB) {
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000332 // If this instruction is a comparison against zero and isn't comparing a
333 // physical register, we can try to optimize it.
334 unsigned SrcReg;
Gabor Greif04ac81d2010-09-21 12:01:15 +0000335 int CmpMask, CmpValue;
336 if (!TII->AnalyzeCompare(MI, SrcReg, CmpMask, CmpValue) ||
Bill Wendling92ad57f2010-09-10 23:34:19 +0000337 TargetRegisterInfo::isPhysicalRegister(SrcReg))
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000338 return false;
339
Bill Wendlinga6556862010-09-11 00:13:50 +0000340 // Attempt to optimize the comparison instruction.
Evan Chengeb96a2f2010-11-15 21:20:45 +0000341 if (TII->OptimizeCompareInstr(MI, SrcReg, CmpMask, CmpValue, MRI)) {
Evan Chengd158fba2011-03-15 05:13:13 +0000342 ++NumCmps;
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000343 return true;
344 }
345
346 return false;
347}
348
Evan Chengc4af4632010-11-17 20:13:28 +0000349bool PeepholeOptimizer::isMoveImmediate(MachineInstr *MI,
350 SmallSet<unsigned, 4> &ImmDefRegs,
351 DenseMap<unsigned, MachineInstr*> &ImmDefMIs) {
Evan Chenge837dea2011-06-28 19:10:37 +0000352 const MCInstrDesc &MCID = MI->getDesc();
Evan Cheng5a96b3d2011-12-07 07:15:52 +0000353 if (!MI->isMoveImmediate())
Evan Chengc4af4632010-11-17 20:13:28 +0000354 return false;
Evan Chenge837dea2011-06-28 19:10:37 +0000355 if (MCID.getNumDefs() != 1)
Evan Chengc4af4632010-11-17 20:13:28 +0000356 return false;
357 unsigned Reg = MI->getOperand(0).getReg();
358 if (TargetRegisterInfo::isVirtualRegister(Reg)) {
359 ImmDefMIs.insert(std::make_pair(Reg, MI));
360 ImmDefRegs.insert(Reg);
361 return true;
362 }
Andrew Trick1df91b02012-02-08 21:22:43 +0000363
Evan Chengc4af4632010-11-17 20:13:28 +0000364 return false;
365}
366
367/// FoldImmediate - Try folding register operands that are defined by move
368/// immediate instructions, i.e. a trivial constant folding optimization, if
369/// and only if the def and use are in the same BB.
370bool PeepholeOptimizer::FoldImmediate(MachineInstr *MI, MachineBasicBlock *MBB,
371 SmallSet<unsigned, 4> &ImmDefRegs,
372 DenseMap<unsigned, MachineInstr*> &ImmDefMIs) {
373 for (unsigned i = 0, e = MI->getDesc().getNumOperands(); i != e; ++i) {
374 MachineOperand &MO = MI->getOperand(i);
375 if (!MO.isReg() || MO.isDef())
376 continue;
377 unsigned Reg = MO.getReg();
Jakob Stoklund Olesenc9df0252011-01-10 02:58:51 +0000378 if (!TargetRegisterInfo::isVirtualRegister(Reg))
Evan Chengc4af4632010-11-17 20:13:28 +0000379 continue;
380 if (ImmDefRegs.count(Reg) == 0)
381 continue;
382 DenseMap<unsigned, MachineInstr*>::iterator II = ImmDefMIs.find(Reg);
383 assert(II != ImmDefMIs.end());
384 if (TII->FoldImmediate(MI, II->second, Reg, MRI)) {
385 ++NumImmFold;
386 return true;
387 }
388 }
389 return false;
390}
391
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000392bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) {
Evan Chengeb96a2f2010-11-15 21:20:45 +0000393 if (DisablePeephole)
394 return false;
Andrew Trick1df91b02012-02-08 21:22:43 +0000395
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000396 TM = &MF.getTarget();
397 TII = TM->getInstrInfo();
398 MRI = &MF.getRegInfo();
399 DT = Aggressive ? &getAnalysis<MachineDominatorTree>() : 0;
400
401 bool Changed = false;
402
403 SmallPtrSet<MachineInstr*, 8> LocalMIs;
Evan Chengc4af4632010-11-17 20:13:28 +0000404 SmallSet<unsigned, 4> ImmDefRegs;
405 DenseMap<unsigned, MachineInstr*> ImmDefMIs;
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000406 for (MachineFunction::iterator I = MF.begin(), E = MF.end(); I != E; ++I) {
407 MachineBasicBlock *MBB = &*I;
Andrew Trick1df91b02012-02-08 21:22:43 +0000408
Evan Chengc4af4632010-11-17 20:13:28 +0000409 bool SeenMoveImm = false;
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000410 LocalMIs.clear();
Evan Chengc4af4632010-11-17 20:13:28 +0000411 ImmDefRegs.clear();
412 ImmDefMIs.clear();
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000413
Evan Cheng326d9762011-02-15 05:00:24 +0000414 bool First = true;
415 MachineBasicBlock::iterator PMII;
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000416 for (MachineBasicBlock::iterator
Bill Wendling220e2402010-09-10 21:55:43 +0000417 MII = I->begin(), MIE = I->end(); MII != MIE; ) {
Evan Chengcf75ab52011-02-14 21:50:37 +0000418 MachineInstr *MI = &*MII;
Evan Chengeb96a2f2010-11-15 21:20:45 +0000419 LocalMIs.insert(MI);
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000420
Evan Cheng30a343a2011-01-07 21:08:26 +0000421 if (MI->isLabel() || MI->isPHI() || MI->isImplicitDef() ||
422 MI->isKill() || MI->isInlineAsm() || MI->isDebugValue() ||
Evan Chengcf75ab52011-02-14 21:50:37 +0000423 MI->hasUnmodeledSideEffects()) {
424 ++MII;
Evan Chengeb96a2f2010-11-15 21:20:45 +0000425 continue;
Evan Chengcf75ab52011-02-14 21:50:37 +0000426 }
Evan Chengeb96a2f2010-11-15 21:20:45 +0000427
Evan Cheng5a96b3d2011-12-07 07:15:52 +0000428 if (MI->isBitcast()) {
Evan Chengd158fba2011-03-15 05:13:13 +0000429 if (OptimizeBitcastInstr(MI, MBB)) {
430 // MI is deleted.
Nick Lewyckydec1b102011-10-13 02:16:18 +0000431 LocalMIs.erase(MI);
Evan Chengd158fba2011-03-15 05:13:13 +0000432 Changed = true;
433 MII = First ? I->begin() : llvm::next(PMII);
434 continue;
Andrew Trick1df91b02012-02-08 21:22:43 +0000435 }
Evan Cheng5a96b3d2011-12-07 07:15:52 +0000436 } else if (MI->isCompare()) {
Evan Chengcf75ab52011-02-14 21:50:37 +0000437 if (OptimizeCmpInstr(MI, MBB)) {
438 // MI is deleted.
Nick Lewyckydec1b102011-10-13 02:16:18 +0000439 LocalMIs.erase(MI);
Evan Chengcf75ab52011-02-14 21:50:37 +0000440 Changed = true;
Evan Cheng326d9762011-02-15 05:00:24 +0000441 MII = First ? I->begin() : llvm::next(PMII);
Evan Chengcf75ab52011-02-14 21:50:37 +0000442 continue;
443 }
444 }
445
446 if (isMoveImmediate(MI, ImmDefRegs, ImmDefMIs)) {
Evan Chengc4af4632010-11-17 20:13:28 +0000447 SeenMoveImm = true;
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000448 } else {
449 Changed |= OptimizeExtInstr(MI, MBB, LocalMIs);
Evan Chengc4af4632010-11-17 20:13:28 +0000450 if (SeenMoveImm)
451 Changed |= FoldImmediate(MI, MBB, ImmDefRegs, ImmDefMIs);
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000452 }
Evan Cheng326d9762011-02-15 05:00:24 +0000453
454 First = false;
Evan Chengcf75ab52011-02-14 21:50:37 +0000455 PMII = MII;
456 ++MII;
Bill Wendling6cdb1ab2010-08-09 23:59:04 +0000457 }
458 }
459
460 return Changed;
461}