Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 1 | //===-- PhiElimination.cpp - Eliminate PHI nodes by inserting copies ------===// |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 2 | // |
| 3 | // 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. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 9 | // |
| 10 | // This pass eliminates machine instruction PHI nodes by inserting copy |
| 11 | // instructions. This destroys SSA information, but is the desired input for |
| 12 | // some register allocators. |
| 13 | // |
| 14 | //===----------------------------------------------------------------------===// |
| 15 | |
Chris Lattner | 0742b59 | 2004-02-23 18:38:20 +0000 | [diff] [blame] | 16 | #include "llvm/CodeGen/Passes.h" |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 17 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 18 | #include "llvm/CodeGen/MachineInstr.h" |
| 19 | #include "llvm/CodeGen/SSARegMap.h" |
| 20 | #include "llvm/CodeGen/LiveVariables.h" |
Chris Lattner | 3501fea | 2003-01-14 22:00:31 +0000 | [diff] [blame] | 21 | #include "llvm/Target/TargetInstrInfo.h" |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 22 | #include "llvm/Target/TargetMachine.h" |
Chris Lattner | 8abf693 | 2004-05-10 19:17:36 +0000 | [diff] [blame] | 23 | #include "Support/DenseMap.h" |
Alkis Evlogimenos | f81af21 | 2004-02-14 01:18:34 +0000 | [diff] [blame] | 24 | #include "Support/STLExtras.h" |
Chris Lattner | 0742b59 | 2004-02-23 18:38:20 +0000 | [diff] [blame] | 25 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 26 | |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 27 | namespace { |
| 28 | struct PNE : public MachineFunctionPass { |
| 29 | bool runOnMachineFunction(MachineFunction &Fn) { |
| 30 | bool Changed = false; |
| 31 | |
| 32 | // Eliminate PHI instructions by inserting copies into predecessor blocks. |
| 33 | // |
| 34 | for (MachineFunction::iterator I = Fn.begin(), E = Fn.end(); I != E; ++I) |
| 35 | Changed |= EliminatePHINodes(Fn, *I); |
| 36 | |
| 37 | //std::cerr << "AFTER PHI NODE ELIM:\n"; |
| 38 | //Fn.dump(); |
| 39 | return Changed; |
| 40 | } |
| 41 | |
| 42 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
| 43 | AU.addPreserved<LiveVariables>(); |
| 44 | MachineFunctionPass::getAnalysisUsage(AU); |
| 45 | } |
| 46 | |
| 47 | private: |
| 48 | /// EliminatePHINodes - Eliminate phi nodes by inserting copy instructions |
| 49 | /// in predecessor basic blocks. |
| 50 | /// |
| 51 | bool EliminatePHINodes(MachineFunction &MF, MachineBasicBlock &MBB); |
| 52 | }; |
| 53 | |
| 54 | RegisterPass<PNE> X("phi-node-elimination", |
| 55 | "Eliminate PHI nodes for register allocation"); |
| 56 | } |
| 57 | |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 58 | |
Chris Lattner | 0742b59 | 2004-02-23 18:38:20 +0000 | [diff] [blame] | 59 | const PassInfo *llvm::PHIEliminationID = X.getPassInfo(); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 60 | |
| 61 | /// EliminatePHINodes - Eliminate phi nodes by inserting copy instructions in |
| 62 | /// predecessor basic blocks. |
| 63 | /// |
| 64 | bool PNE::EliminatePHINodes(MachineFunction &MF, MachineBasicBlock &MBB) { |
Alkis Evlogimenos | c0b9dc5 | 2004-02-12 02:27:10 +0000 | [diff] [blame] | 65 | if (MBB.empty() || MBB.front().getOpcode() != TargetInstrInfo::PHI) |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 66 | return false; // Quick exit for normal case... |
| 67 | |
| 68 | LiveVariables *LV = getAnalysisToUpdate<LiveVariables>(); |
| 69 | const TargetInstrInfo &MII = MF.getTarget().getInstrInfo(); |
| 70 | const MRegisterInfo *RegInfo = MF.getTarget().getRegisterInfo(); |
| 71 | |
Chris Lattner | 80e20eb | 2004-05-10 19:06:37 +0000 | [diff] [blame] | 72 | // VRegPHIUseCount - Keep track of the number of times each virtual register |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 73 | // is used by PHI nodes in successors of this block. |
Chris Lattner | 8abf693 | 2004-05-10 19:17:36 +0000 | [diff] [blame] | 74 | DenseMap<unsigned, VirtReg2IndexFunctor> VRegPHIUseCount; |
| 75 | VRegPHIUseCount.grow(MF.getSSARegMap()->getLastVirtReg()); |
Chris Lattner | 80e20eb | 2004-05-10 19:06:37 +0000 | [diff] [blame] | 76 | |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 77 | unsigned BBIsSuccOfPreds = 0; // Number of times MBB is a succ of preds |
| 78 | for (MachineBasicBlock::pred_iterator PI = MBB.pred_begin(), |
| 79 | E = MBB.pred_end(); PI != E; ++PI) |
| 80 | for (MachineBasicBlock::succ_iterator SI = (*PI)->succ_begin(), |
| 81 | E = (*PI)->succ_end(); SI != E; ++SI) { |
| 82 | BBIsSuccOfPreds += *SI == &MBB; |
| 83 | for (MachineBasicBlock::iterator BBI = (*SI)->begin(); BBI !=(*SI)->end() && |
| 84 | BBI->getOpcode() == TargetInstrInfo::PHI; ++BBI) |
| 85 | for (unsigned i = 1, e = BBI->getNumOperands(); i != e; i += 2) |
| 86 | VRegPHIUseCount[BBI->getOperand(i).getReg()]++; |
| 87 | } |
| 88 | |
Chris Lattner | 791f896 | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 89 | // Get an iterator to the first instruction after the last PHI node (this may |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 90 | // also be the end of the basic block). While we are scanning the PHIs, |
Chris Lattner | 80e20eb | 2004-05-10 19:06:37 +0000 | [diff] [blame] | 91 | // populate the VRegPHIUseCount map. |
Chris Lattner | 791f896 | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 92 | MachineBasicBlock::iterator AfterPHIsIt = MBB.begin(); |
| 93 | while (AfterPHIsIt != MBB.end() && |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 94 | AfterPHIsIt->getOpcode() == TargetInstrInfo::PHI) |
Chris Lattner | 791f896 | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 95 | ++AfterPHIsIt; // Skip over all of the PHI nodes... |
| 96 | |
Alkis Evlogimenos | c0b9dc5 | 2004-02-12 02:27:10 +0000 | [diff] [blame] | 97 | while (MBB.front().getOpcode() == TargetInstrInfo::PHI) { |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 98 | // Unlink the PHI node from the basic block... but don't delete the PHI yet |
Chris Lattner | 4f6410f | 2004-03-31 21:59:29 +0000 | [diff] [blame] | 99 | MachineInstr *MI = MBB.remove(MBB.begin()); |
Alkis Evlogimenos | c0b9dc5 | 2004-02-12 02:27:10 +0000 | [diff] [blame] | 100 | |
Chris Lattner | 1cbe4d0 | 2004-02-10 21:12:22 +0000 | [diff] [blame] | 101 | assert(MRegisterInfo::isVirtualRegister(MI->getOperand(0).getReg()) && |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 102 | "PHI node doesn't write virt reg?"); |
| 103 | |
Alkis Evlogimenos | be766c7 | 2004-02-13 21:01:20 +0000 | [diff] [blame] | 104 | unsigned DestReg = MI->getOperand(0).getReg(); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 105 | |
| 106 | // Create a new register for the incoming PHI arguments |
| 107 | const TargetRegisterClass *RC = MF.getSSARegMap()->getRegClass(DestReg); |
| 108 | unsigned IncomingReg = MF.getSSARegMap()->createVirtualRegister(RC); |
| 109 | |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 110 | // Insert a register to register copy in the top of the current block (but |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 111 | // after any remaining phi nodes) which copies the new incoming register |
| 112 | // into the phi node destination. |
| 113 | // |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 114 | RegInfo->copyRegToReg(MBB, AfterPHIsIt, DestReg, IncomingReg, RC); |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 115 | |
| 116 | // Update live variable information if there is any... |
| 117 | if (LV) { |
Chris Lattner | 791f896 | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 118 | MachineInstr *PHICopy = prior(AfterPHIsIt); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 119 | |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 120 | // Add information to LiveVariables to know that the incoming value is |
Chris Lattner | b52e024 | 2003-05-12 17:37:30 +0000 | [diff] [blame] | 121 | // killed. Note that because the value is defined in several places (once |
| 122 | // each for each incoming block), the "def" block and instruction fields |
| 123 | // for the VarInfo is not filled in. |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 124 | // |
Chris Lattner | b52e024 | 2003-05-12 17:37:30 +0000 | [diff] [blame] | 125 | LV->addVirtualRegisterKilled(IncomingReg, &MBB, PHICopy); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 126 | |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 127 | // Since we are going to be deleting the PHI node, if it is the last use |
| 128 | // of any registers, or if the value itself is dead, we need to move this |
| 129 | // information over to the new copy we just inserted... |
| 130 | // |
| 131 | std::pair<LiveVariables::killed_iterator, LiveVariables::killed_iterator> |
| 132 | RKs = LV->killed_range(MI); |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 133 | std::vector<std::pair<MachineInstr*, unsigned> > Range; |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 134 | if (RKs.first != RKs.second) { |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 135 | // Copy the range into a vector... |
| 136 | Range.assign(RKs.first, RKs.second); |
| 137 | |
| 138 | // Delete the range... |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 139 | LV->removeVirtualRegistersKilled(RKs.first, RKs.second); |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 140 | |
| 141 | // Add all of the kills back, which will update the appropriate info... |
| 142 | for (unsigned i = 0, e = Range.size(); i != e; ++i) |
| 143 | LV->addVirtualRegisterKilled(Range[i].second, &MBB, PHICopy); |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 144 | } |
| 145 | |
| 146 | RKs = LV->dead_range(MI); |
| 147 | if (RKs.first != RKs.second) { |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 148 | // Works as above... |
| 149 | Range.assign(RKs.first, RKs.second); |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 150 | LV->removeVirtualRegistersDead(RKs.first, RKs.second); |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 151 | for (unsigned i = 0, e = Range.size(); i != e; ++i) |
| 152 | LV->addVirtualRegisterDead(Range[i].second, &MBB, PHICopy); |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 153 | } |
| 154 | } |
| 155 | |
Chris Lattner | 80e20eb | 2004-05-10 19:06:37 +0000 | [diff] [blame] | 156 | // Adjust the VRegPHIUseCount map to account for the removal of this PHI |
| 157 | // node. |
| 158 | for (unsigned i = 1; i != MI->getNumOperands(); i += 2) |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 159 | VRegPHIUseCount[MI->getOperand(i).getReg()] -= BBIsSuccOfPreds; |
Chris Lattner | 80e20eb | 2004-05-10 19:06:37 +0000 | [diff] [blame] | 160 | |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 161 | // Now loop over all of the incoming arguments, changing them to copy into |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 162 | // the IncomingReg register in the corresponding predecessor basic block. |
| 163 | // |
| 164 | for (int i = MI->getNumOperands() - 1; i >= 2; i-=2) { |
| 165 | MachineOperand &opVal = MI->getOperand(i-1); |
| 166 | |
| 167 | // Get the MachineBasicBlock equivalent of the BasicBlock that is the |
Chris Lattner | 927ce5d | 2003-05-12 03:55:21 +0000 | [diff] [blame] | 168 | // source path the PHI. |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 169 | MachineBasicBlock &opBlock = *MI->getOperand(i).getMachineBasicBlock(); |
| 170 | |
Alkis Evlogimenos | 743d0a1 | 2004-02-23 18:14:48 +0000 | [diff] [blame] | 171 | MachineBasicBlock::iterator I = opBlock.getFirstTerminator(); |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 172 | |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 173 | // Check to make sure we haven't already emitted the copy for this block. |
| 174 | // This can happen because PHI nodes may have multiple entries for the |
| 175 | // same basic block. It doesn't matter which entry we use though, because |
| 176 | // all incoming values are guaranteed to be the same for a particular bb. |
| 177 | // |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 178 | // If we emitted a copy for this basic block already, it will be right |
| 179 | // where we want to insert one now. Just check for a definition of the |
| 180 | // register we are interested in! |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 181 | // |
| 182 | bool HaveNotEmitted = true; |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 183 | |
| 184 | if (I != opBlock.begin()) { |
Alkis Evlogimenos | f81af21 | 2004-02-14 01:18:34 +0000 | [diff] [blame] | 185 | MachineBasicBlock::iterator PrevInst = prior(I); |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 186 | for (unsigned i = 0, e = PrevInst->getNumOperands(); i != e; ++i) { |
| 187 | MachineOperand &MO = PrevInst->getOperand(i); |
Chris Lattner | 1cbe4d0 | 2004-02-10 21:12:22 +0000 | [diff] [blame] | 188 | if (MO.isRegister() && MO.getReg() == IncomingReg) |
Alkis Evlogimenos | 4d7af65 | 2003-12-14 13:24:17 +0000 | [diff] [blame] | 189 | if (MO.isDef()) { |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 190 | HaveNotEmitted = false; |
| 191 | break; |
| 192 | } |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 193 | } |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 194 | } |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 195 | |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 196 | if (HaveNotEmitted) { // If the copy has not already been emitted, do it. |
Chris Lattner | 1cbe4d0 | 2004-02-10 21:12:22 +0000 | [diff] [blame] | 197 | assert(MRegisterInfo::isVirtualRegister(opVal.getReg()) && |
Chris Lattner | 98719d7 | 2003-05-12 04:08:54 +0000 | [diff] [blame] | 198 | "Machine PHI Operands must all be virtual registers!"); |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 199 | unsigned SrcReg = opVal.getReg(); |
| 200 | RegInfo->copyRegToReg(opBlock, I, IncomingReg, SrcReg, RC); |
| 201 | |
| 202 | // Now update live variable information if we have it. |
| 203 | if (LV) { |
| 204 | // We want to be able to insert a kill of the register if this PHI |
| 205 | // (aka, the copy we just inserted) is the last use of the source |
| 206 | // value. Live variable analysis conservatively handles this by |
| 207 | // saying that the value is live until the end of the block the PHI |
| 208 | // entry lives in. If the value really is dead at the PHI copy, there |
| 209 | // will be no successor blocks which have the value live-in. |
| 210 | // |
| 211 | // Check to see if the copy is the last use, and if so, update the |
| 212 | // live variables information so that it knows the copy source |
| 213 | // instruction kills the incoming value. |
| 214 | // |
| 215 | LiveVariables::VarInfo &InRegVI = LV->getVarInfo(SrcReg); |
| 216 | |
Chris Lattner | 08d2e4e | 2003-06-05 17:15:04 +0000 | [diff] [blame] | 217 | // Loop over all of the successors of the basic block, checking to see |
| 218 | // if the value is either live in the block, or if it is killed in the |
| 219 | // block. Also check to see if this register is in use by another PHI |
| 220 | // node which has not yet been eliminated. If so, it will be killed |
| 221 | // at an appropriate point later. |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 222 | // |
| 223 | bool ValueIsLive = false; |
Chris Lattner | 015959e | 2004-05-01 21:24:39 +0000 | [diff] [blame] | 224 | for (MachineBasicBlock::succ_iterator SI = opBlock.succ_begin(), |
| 225 | E = opBlock.succ_end(); SI != E && !ValueIsLive; ++SI) { |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 226 | MachineBasicBlock *SuccMBB = *SI; |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 227 | |
| 228 | // Is it alive in this successor? |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 229 | unsigned SuccIdx = LV->getMachineBasicBlockIndex(SuccMBB); |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 230 | if (SuccIdx < InRegVI.AliveBlocks.size() && |
| 231 | InRegVI.AliveBlocks[SuccIdx]) { |
| 232 | ValueIsLive = true; |
| 233 | break; |
| 234 | } |
| 235 | |
| 236 | // Is it killed in this successor? |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 237 | for (unsigned i = 0, e = InRegVI.Kills.size(); i != e; ++i) |
Chris Lattner | bee8872 | 2004-05-12 21:47:57 +0000 | [diff] [blame^] | 238 | if (InRegVI.Kills[i].first == SuccMBB) { |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 239 | ValueIsLive = true; |
| 240 | break; |
| 241 | } |
Chris Lattner | 08d2e4e | 2003-06-05 17:15:04 +0000 | [diff] [blame] | 242 | |
| 243 | // Is it used by any PHI instructions in this block? |
Chris Lattner | 8abf693 | 2004-05-10 19:17:36 +0000 | [diff] [blame] | 244 | if (!ValueIsLive) |
| 245 | ValueIsLive = VRegPHIUseCount[SrcReg] != 0; |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 246 | } |
| 247 | |
| 248 | // Okay, if we now know that the value is not live out of the block, |
| 249 | // we can add a kill marker to the copy we inserted saying that it |
| 250 | // kills the incoming value! |
| 251 | // |
Alkis Evlogimenos | c0b9dc5 | 2004-02-12 02:27:10 +0000 | [diff] [blame] | 252 | if (!ValueIsLive) { |
Alkis Evlogimenos | f81af21 | 2004-02-14 01:18:34 +0000 | [diff] [blame] | 253 | MachineBasicBlock::iterator Prev = prior(I); |
Alkis Evlogimenos | c0b9dc5 | 2004-02-12 02:27:10 +0000 | [diff] [blame] | 254 | LV->addVirtualRegisterKilled(SrcReg, &opBlock, Prev); |
| 255 | } |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 256 | } |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 257 | } |
| 258 | } |
| 259 | |
| 260 | // really delete the PHI instruction now! |
| 261 | delete MI; |
| 262 | } |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 263 | return true; |
| 264 | } |