Jia Liu | b22310f | 2012-02-18 12:03:15 +0000 | [diff] [blame] | 1 | //===-- PPCBranchSelector.cpp - Emit long conditional branches ------------===// |
Misha Brukman | b440243 | 2005-04-21 23:30:14 +0000 | [diff] [blame] | 2 | // |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | b440243 | 2005-04-21 23:30:14 +0000 | [diff] [blame] | 7 | // |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
Misha Brukman | b440243 | 2005-04-21 23:30:14 +0000 | [diff] [blame] | 10 | // This file contains a pass that scans a machine function to determine which |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 11 | // conditional branches need more than 16 bits of displacement to reach their |
| 12 | // target basic block. It does this in two passes; a calculation of basic block |
Gabor Greif | 21fed66 | 2010-08-23 20:30:51 +0000 | [diff] [blame] | 13 | // positions pass, and a branch pseudo op to machine branch opcode pass. This |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 14 | // pass should be run last, just before the assembly printer. |
| 15 | // |
| 16 | //===----------------------------------------------------------------------===// |
| 17 | |
Chris Lattner | bfca1ab | 2005-10-14 23:51:18 +0000 | [diff] [blame] | 18 | #include "PPC.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 19 | #include "MCTargetDesc/PPCPredicates.h" |
Chris Lattner | e80bf1b | 2005-10-14 23:45:43 +0000 | [diff] [blame] | 20 | #include "PPCInstrBuilder.h" |
Chris Lattner | 6f3b954 | 2005-10-14 23:59:06 +0000 | [diff] [blame] | 21 | #include "PPCInstrInfo.h" |
Hal Finkel | 530fa5f | 2016-10-03 04:06:44 +0000 | [diff] [blame] | 22 | #include "PPCSubtarget.h" |
Chris Lattner | 96d7386 | 2006-11-16 18:13:49 +0000 | [diff] [blame] | 23 | #include "llvm/ADT/Statistic.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 24 | #include "llvm/CodeGen/MachineFunctionPass.h" |
Hal Finkel | 530fa5f | 2016-10-03 04:06:44 +0000 | [diff] [blame] | 25 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 26 | #include "llvm/Support/MathExtras.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 27 | #include "llvm/Target/TargetMachine.h" |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 28 | #include "llvm/Target/TargetSubtargetInfo.h" |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 29 | using namespace llvm; |
| 30 | |
Chandler Carruth | 84e68b2 | 2014-04-22 02:41:26 +0000 | [diff] [blame] | 31 | #define DEBUG_TYPE "ppc-branch-select" |
| 32 | |
Chris Lattner | 1ef9cd4 | 2006-12-19 22:59:26 +0000 | [diff] [blame] | 33 | STATISTIC(NumExpanded, "Number of branches expanded to long format"); |
Chris Lattner | 96d7386 | 2006-11-16 18:13:49 +0000 | [diff] [blame] | 34 | |
Krzysztof Parzyszek | 2680b53 | 2013-02-13 17:40:07 +0000 | [diff] [blame] | 35 | namespace llvm { |
| 36 | void initializePPCBSelPass(PassRegistry&); |
| 37 | } |
| 38 | |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 39 | namespace { |
Nick Lewycky | 02d5f77 | 2009-10-25 06:33:48 +0000 | [diff] [blame] | 40 | struct PPCBSel : public MachineFunctionPass { |
Devang Patel | 8c78a0b | 2007-05-03 01:11:54 +0000 | [diff] [blame] | 41 | static char ID; |
Krzysztof Parzyszek | 2680b53 | 2013-02-13 17:40:07 +0000 | [diff] [blame] | 42 | PPCBSel() : MachineFunctionPass(ID) { |
| 43 | initializePPCBSelPass(*PassRegistry::getPassRegistry()); |
| 44 | } |
Devang Patel | 09f162c | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 45 | |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 46 | // The sizes of the basic blocks in the function (the first |
| 47 | // element of the pair); the second element of the pair is the amount of the |
| 48 | // size that is due to potential padding. |
| 49 | std::vector<std::pair<unsigned, unsigned>> BlockSizes; |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 50 | |
Craig Topper | 0d3fa92 | 2014-04-29 07:57:37 +0000 | [diff] [blame] | 51 | bool runOnMachineFunction(MachineFunction &Fn) override; |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 52 | |
Derek Schuff | 1dbf7a5 | 2016-04-04 17:09:25 +0000 | [diff] [blame] | 53 | MachineFunctionProperties getRequiredProperties() const override { |
| 54 | return MachineFunctionProperties().set( |
Matthias Braun | 1eb4736 | 2016-08-25 01:27:13 +0000 | [diff] [blame] | 55 | MachineFunctionProperties::Property::NoVRegs); |
Derek Schuff | 1dbf7a5 | 2016-04-04 17:09:25 +0000 | [diff] [blame] | 56 | } |
| 57 | |
Mehdi Amini | 117296c | 2016-10-01 02:56:57 +0000 | [diff] [blame] | 58 | StringRef getPassName() const override { return "PowerPC Branch Selector"; } |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 59 | }; |
Devang Patel | 8c78a0b | 2007-05-03 01:11:54 +0000 | [diff] [blame] | 60 | char PPCBSel::ID = 0; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 61 | } |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 62 | |
Krzysztof Parzyszek | 2680b53 | 2013-02-13 17:40:07 +0000 | [diff] [blame] | 63 | INITIALIZE_PASS(PPCBSel, "ppc-branch-select", "PowerPC Branch Selector", |
| 64 | false, false) |
| 65 | |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 66 | /// createPPCBranchSelectionPass - returns an instance of the Branch Selection |
| 67 | /// Pass |
| 68 | /// |
| 69 | FunctionPass *llvm::createPPCBranchSelectionPass() { |
Chris Lattner | 26e385a | 2006-02-08 19:33:26 +0000 | [diff] [blame] | 70 | return new PPCBSel(); |
Misha Brukman | ef8cf02 | 2004-07-27 18:33:06 +0000 | [diff] [blame] | 71 | } |
Chris Lattner | 26e385a | 2006-02-08 19:33:26 +0000 | [diff] [blame] | 72 | |
Chris Lattner | 26e385a | 2006-02-08 19:33:26 +0000 | [diff] [blame] | 73 | bool PPCBSel::runOnMachineFunction(MachineFunction &Fn) { |
Eric Christopher | fc6de42 | 2014-08-05 02:39:49 +0000 | [diff] [blame] | 74 | const PPCInstrInfo *TII = |
| 75 | static_cast<const PPCInstrInfo *>(Fn.getSubtarget().getInstrInfo()); |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 76 | // Give the blocks of the function a dense, in-order, numbering. |
| 77 | Fn.RenumberBlocks(); |
| 78 | BlockSizes.resize(Fn.getNumBlockIDs()); |
| 79 | |
Hal Finkel | d73bfba | 2015-01-03 14:58:25 +0000 | [diff] [blame] | 80 | auto GetAlignmentAdjustment = |
| 81 | [TII](MachineBasicBlock &MBB, unsigned Offset) -> unsigned { |
| 82 | unsigned Align = MBB.getAlignment(); |
| 83 | if (!Align) |
| 84 | return 0; |
| 85 | |
| 86 | unsigned AlignAmt = 1 << Align; |
| 87 | unsigned ParentAlign = MBB.getParent()->getAlignment(); |
| 88 | |
| 89 | if (Align <= ParentAlign) |
| 90 | return OffsetToAlignment(Offset, AlignAmt); |
| 91 | |
| 92 | // The alignment of this MBB is larger than the function's alignment, so we |
| 93 | // can't tell whether or not it will insert nops. Assume that it will. |
| 94 | return AlignAmt + OffsetToAlignment(Offset, AlignAmt); |
| 95 | }; |
| 96 | |
Hal Finkel | 530fa5f | 2016-10-03 04:06:44 +0000 | [diff] [blame] | 97 | // We need to be careful about the offset of the first block in the function |
| 98 | // because it might not have the function's alignment. This happens because, |
| 99 | // under the ELFv2 ABI, for functions which require a TOC pointer, we add a |
| 100 | // two-instruction sequence to the start of the function. |
| 101 | // Note: This needs to be synchronized with the check in |
| 102 | // PPCLinuxAsmPrinter::EmitFunctionBodyStart. |
| 103 | unsigned InitialOffset = 0; |
| 104 | if (Fn.getSubtarget<PPCSubtarget>().isELFv2ABI() && |
| 105 | !Fn.getRegInfo().use_empty(PPC::X2)) |
| 106 | InitialOffset = 8; |
| 107 | |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 108 | // Measure each MBB and compute a size for the entire function. |
Hal Finkel | 530fa5f | 2016-10-03 04:06:44 +0000 | [diff] [blame] | 109 | unsigned FuncSize = InitialOffset; |
Chris Lattner | 26e385a | 2006-02-08 19:33:26 +0000 | [diff] [blame] | 110 | for (MachineFunction::iterator MFI = Fn.begin(), E = Fn.end(); MFI != E; |
| 111 | ++MFI) { |
Duncan P. N. Exon Smith | ac65b4c | 2015-10-20 01:07:37 +0000 | [diff] [blame] | 112 | MachineBasicBlock *MBB = &*MFI; |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 113 | |
Hal Finkel | d73bfba | 2015-01-03 14:58:25 +0000 | [diff] [blame] | 114 | // The end of the previous block may have extra nops if this block has an |
| 115 | // alignment requirement. |
| 116 | if (MBB->getNumber() > 0) { |
| 117 | unsigned AlignExtra = GetAlignmentAdjustment(*MBB, FuncSize); |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 118 | |
| 119 | auto &BS = BlockSizes[MBB->getNumber()-1]; |
| 120 | BS.first += AlignExtra; |
| 121 | BS.second = AlignExtra; |
| 122 | |
Hal Finkel | d73bfba | 2015-01-03 14:58:25 +0000 | [diff] [blame] | 123 | FuncSize += AlignExtra; |
| 124 | } |
| 125 | |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 126 | unsigned BlockSize = 0; |
Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 127 | for (MachineInstr &MI : *MBB) |
Sjoerd Meijer | 89217f8 | 2016-07-28 16:32:22 +0000 | [diff] [blame] | 128 | BlockSize += TII->getInstSizeInBytes(MI); |
Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 129 | |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 130 | BlockSizes[MBB->getNumber()].first = BlockSize; |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 131 | FuncSize += BlockSize; |
Chris Lattner | 26e385a | 2006-02-08 19:33:26 +0000 | [diff] [blame] | 132 | } |
| 133 | |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 134 | // If the entire function is smaller than the displacement of a branch field, |
| 135 | // we know we don't need to shrink any branches in this function. This is a |
| 136 | // common case. |
| 137 | if (FuncSize < (1 << 15)) { |
| 138 | BlockSizes.clear(); |
| 139 | return false; |
| 140 | } |
| 141 | |
| 142 | // For each conditional branch, if the offset to its destination is larger |
| 143 | // than the offset field allows, transform it into a long branch sequence |
| 144 | // like this: |
| 145 | // short branch: |
| 146 | // bCC MBB |
| 147 | // long branch: |
| 148 | // b!CC $PC+8 |
| 149 | // b MBB |
| 150 | // |
| 151 | bool MadeChange = true; |
| 152 | bool EverMadeChange = false; |
| 153 | while (MadeChange) { |
| 154 | // Iteratively expand branches until we reach a fixed point. |
| 155 | MadeChange = false; |
| 156 | |
| 157 | for (MachineFunction::iterator MFI = Fn.begin(), E = Fn.end(); MFI != E; |
| 158 | ++MFI) { |
| 159 | MachineBasicBlock &MBB = *MFI; |
| 160 | unsigned MBBStartOffset = 0; |
| 161 | for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end(); |
| 162 | I != E; ++I) { |
Craig Topper | 062a2ba | 2014-04-25 05:30:21 +0000 | [diff] [blame] | 163 | MachineBasicBlock *Dest = nullptr; |
Hal Finkel | c521129 | 2013-05-21 14:21:09 +0000 | [diff] [blame] | 164 | if (I->getOpcode() == PPC::BCC && !I->getOperand(2).isImm()) |
| 165 | Dest = I->getOperand(2).getMBB(); |
Hal Finkel | 940ab93 | 2014-02-28 00:27:01 +0000 | [diff] [blame] | 166 | else if ((I->getOpcode() == PPC::BC || I->getOpcode() == PPC::BCn) && |
| 167 | !I->getOperand(1).isImm()) |
| 168 | Dest = I->getOperand(1).getMBB(); |
Hal Finkel | c521129 | 2013-05-21 14:21:09 +0000 | [diff] [blame] | 169 | else if ((I->getOpcode() == PPC::BDNZ8 || I->getOpcode() == PPC::BDNZ || |
| 170 | I->getOpcode() == PPC::BDZ8 || I->getOpcode() == PPC::BDZ) && |
| 171 | !I->getOperand(0).isImm()) |
| 172 | Dest = I->getOperand(0).getMBB(); |
| 173 | |
| 174 | if (!Dest) { |
Sjoerd Meijer | 89217f8 | 2016-07-28 16:32:22 +0000 | [diff] [blame] | 175 | MBBStartOffset += TII->getInstSizeInBytes(*I); |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 176 | continue; |
| 177 | } |
| 178 | |
| 179 | // Determine the offset from the current branch to the destination |
| 180 | // block. |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 181 | int BranchSize; |
| 182 | if (Dest->getNumber() <= MBB.getNumber()) { |
| 183 | // If this is a backwards branch, the delta is the offset from the |
| 184 | // start of this block to this branch, plus the sizes of all blocks |
| 185 | // from this block to the dest. |
| 186 | BranchSize = MBBStartOffset; |
| 187 | |
| 188 | for (unsigned i = Dest->getNumber(), e = MBB.getNumber(); i != e; ++i) |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 189 | BranchSize += BlockSizes[i].first; |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 190 | } else { |
| 191 | // Otherwise, add the size of the blocks between this block and the |
| 192 | // dest to the number of bytes left in this block. |
| 193 | BranchSize = -MBBStartOffset; |
| 194 | |
| 195 | for (unsigned i = MBB.getNumber(), e = Dest->getNumber(); i != e; ++i) |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 196 | BranchSize += BlockSizes[i].first; |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 197 | } |
| 198 | |
| 199 | // If this branch is in range, ignore it. |
Benjamin Kramer | 2788f79 | 2010-03-29 21:13:41 +0000 | [diff] [blame] | 200 | if (isInt<16>(BranchSize)) { |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 201 | MBBStartOffset += 4; |
| 202 | continue; |
| 203 | } |
Hal Finkel | 96c2d4d | 2012-06-08 15:38:21 +0000 | [diff] [blame] | 204 | |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 205 | // Otherwise, we have to expand it to a long branch. |
Duncan P. N. Exon Smith | e5a22f4 | 2016-07-27 13:24:16 +0000 | [diff] [blame] | 206 | MachineInstr &OldBranch = *I; |
| 207 | DebugLoc dl = OldBranch.getDebugLoc(); |
| 208 | |
Hal Finkel | 96c2d4d | 2012-06-08 15:38:21 +0000 | [diff] [blame] | 209 | if (I->getOpcode() == PPC::BCC) { |
| 210 | // The BCC operands are: |
| 211 | // 0. PPC branch predicate |
| 212 | // 1. CR register |
| 213 | // 2. Target MBB |
| 214 | PPC::Predicate Pred = (PPC::Predicate)I->getOperand(0).getImm(); |
| 215 | unsigned CRReg = I->getOperand(1).getReg(); |
| 216 | |
| 217 | // Jump over the uncond branch inst (i.e. $PC+8) on opposite condition. |
| 218 | BuildMI(MBB, I, dl, TII->get(PPC::BCC)) |
| 219 | .addImm(PPC::InvertPredicate(Pred)).addReg(CRReg).addImm(2); |
Hal Finkel | 940ab93 | 2014-02-28 00:27:01 +0000 | [diff] [blame] | 220 | } else if (I->getOpcode() == PPC::BC) { |
| 221 | unsigned CRBit = I->getOperand(0).getReg(); |
| 222 | BuildMI(MBB, I, dl, TII->get(PPC::BCn)).addReg(CRBit).addImm(2); |
| 223 | } else if (I->getOpcode() == PPC::BCn) { |
| 224 | unsigned CRBit = I->getOperand(0).getReg(); |
| 225 | BuildMI(MBB, I, dl, TII->get(PPC::BC)).addReg(CRBit).addImm(2); |
Hal Finkel | 96c2d4d | 2012-06-08 15:38:21 +0000 | [diff] [blame] | 226 | } else if (I->getOpcode() == PPC::BDNZ) { |
| 227 | BuildMI(MBB, I, dl, TII->get(PPC::BDZ)).addImm(2); |
| 228 | } else if (I->getOpcode() == PPC::BDNZ8) { |
| 229 | BuildMI(MBB, I, dl, TII->get(PPC::BDZ8)).addImm(2); |
| 230 | } else if (I->getOpcode() == PPC::BDZ) { |
| 231 | BuildMI(MBB, I, dl, TII->get(PPC::BDNZ)).addImm(2); |
| 232 | } else if (I->getOpcode() == PPC::BDZ8) { |
| 233 | BuildMI(MBB, I, dl, TII->get(PPC::BDNZ8)).addImm(2); |
| 234 | } else { |
| 235 | llvm_unreachable("Unhandled branch type!"); |
| 236 | } |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 237 | |
| 238 | // Uncond branch to the real destination. |
Dale Johannesen | e9f623e | 2009-02-13 02:27:39 +0000 | [diff] [blame] | 239 | I = BuildMI(MBB, I, dl, TII->get(PPC::B)).addMBB(Dest); |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 240 | |
| 241 | // Remove the old branch from the function. |
Duncan P. N. Exon Smith | e5a22f4 | 2016-07-27 13:24:16 +0000 | [diff] [blame] | 242 | OldBranch.eraseFromParent(); |
| 243 | |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 244 | // Remember that this instruction is 8-bytes, increase the size of the |
| 245 | // block by 4, remember to iterate. |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 246 | BlockSizes[MBB.getNumber()].first += 4; |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 247 | MBBStartOffset += 8; |
| 248 | ++NumExpanded; |
| 249 | MadeChange = true; |
| 250 | } |
| 251 | } |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 252 | |
| 253 | if (MadeChange) { |
| 254 | // If we're going to iterate again, make sure we've updated our |
| 255 | // padding-based contributions to the block sizes. |
Hal Finkel | 530fa5f | 2016-10-03 04:06:44 +0000 | [diff] [blame] | 256 | unsigned Offset = InitialOffset; |
Hal Finkel | f0bc9db | 2016-09-04 14:18:29 +0000 | [diff] [blame] | 257 | for (MachineFunction::iterator MFI = Fn.begin(), E = Fn.end(); MFI != E; |
| 258 | ++MFI) { |
| 259 | MachineBasicBlock *MBB = &*MFI; |
| 260 | |
| 261 | if (MBB->getNumber() > 0) { |
| 262 | auto &BS = BlockSizes[MBB->getNumber()-1]; |
| 263 | BS.first -= BS.second; |
| 264 | Offset -= BS.second; |
| 265 | |
| 266 | unsigned AlignExtra = GetAlignmentAdjustment(*MBB, Offset); |
| 267 | |
| 268 | BS.first += AlignExtra; |
| 269 | BS.second = AlignExtra; |
| 270 | |
| 271 | Offset += AlignExtra; |
| 272 | } |
| 273 | |
| 274 | Offset += BlockSizes[MBB->getNumber()].first; |
| 275 | } |
| 276 | } |
| 277 | |
Chris Lattner | 542dfd5 | 2006-11-18 00:32:03 +0000 | [diff] [blame] | 278 | EverMadeChange |= MadeChange; |
| 279 | } |
| 280 | |
| 281 | BlockSizes.clear(); |
Chris Lattner | 26e385a | 2006-02-08 19:33:26 +0000 | [diff] [blame] | 282 | return true; |
| 283 | } |