Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 1 | //===-- GCRootLowering.cpp - Garbage collection infrastructure ------------===// |
| 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 | // This file implements the lowering for the gc.root mechanism. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
Philip Reames | 2b45395 | 2015-01-16 20:07:33 +0000 | [diff] [blame] | 14 | #include "llvm/CodeGen/GCMetadata.h" |
Chandler Carruth | 71f308a | 2015-02-13 09:09:03 +0000 | [diff] [blame] | 15 | #include "llvm/CodeGen/GCStrategy.h" |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 16 | #include "llvm/CodeGen/MachineFrameInfo.h" |
| 17 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 18 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
| 19 | #include "llvm/CodeGen/MachineModuleInfo.h" |
| 20 | #include "llvm/CodeGen/Passes.h" |
| 21 | #include "llvm/IR/Dominators.h" |
| 22 | #include "llvm/IR/IntrinsicInst.h" |
| 23 | #include "llvm/IR/Module.h" |
| 24 | #include "llvm/Support/Debug.h" |
| 25 | #include "llvm/Support/ErrorHandling.h" |
| 26 | #include "llvm/Support/raw_ostream.h" |
| 27 | #include "llvm/Target/TargetFrameLowering.h" |
| 28 | #include "llvm/Target/TargetInstrInfo.h" |
| 29 | #include "llvm/Target/TargetMachine.h" |
| 30 | #include "llvm/Target/TargetRegisterInfo.h" |
| 31 | #include "llvm/Target/TargetSubtargetInfo.h" |
| 32 | |
| 33 | using namespace llvm; |
| 34 | |
| 35 | namespace { |
| 36 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 37 | /// LowerIntrinsics - This pass rewrites calls to the llvm.gcread or |
| 38 | /// llvm.gcwrite intrinsics, replacing them with simple loads and stores as |
| 39 | /// directed by the GCStrategy. It also performs automatic root initialization |
| 40 | /// and custom intrinsic lowering. |
| 41 | class LowerIntrinsics : public FunctionPass { |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 42 | bool PerformDefaultLowering(Function &F, GCStrategy &Coll); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 43 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 44 | public: |
| 45 | static char ID; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 46 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 47 | LowerIntrinsics(); |
| 48 | const char *getPassName() const override; |
| 49 | void getAnalysisUsage(AnalysisUsage &AU) const override; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 50 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 51 | bool doInitialization(Module &M) override; |
| 52 | bool runOnFunction(Function &F) override; |
| 53 | }; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 54 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 55 | /// GCMachineCodeAnalysis - This is a target-independent pass over the machine |
| 56 | /// function representation to identify safe points for the garbage collector |
| 57 | /// in the machine code. It inserts labels at safe points and populates a |
| 58 | /// GCMetadata record for each function. |
| 59 | class GCMachineCodeAnalysis : public MachineFunctionPass { |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 60 | GCFunctionInfo *FI; |
| 61 | MachineModuleInfo *MMI; |
| 62 | const TargetInstrInfo *TII; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 63 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 64 | void FindSafePoints(MachineFunction &MF); |
| 65 | void VisitCallPoint(MachineBasicBlock::iterator MI); |
| 66 | MCSymbol *InsertLabel(MachineBasicBlock &MBB, MachineBasicBlock::iterator MI, |
| 67 | DebugLoc DL) const; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 68 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 69 | void FindStackOffsets(MachineFunction &MF); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 70 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 71 | public: |
| 72 | static char ID; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 73 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 74 | GCMachineCodeAnalysis(); |
| 75 | void getAnalysisUsage(AnalysisUsage &AU) const override; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 76 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 77 | bool runOnMachineFunction(MachineFunction &MF) override; |
| 78 | }; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 79 | } |
| 80 | |
| 81 | // ----------------------------------------------------------------------------- |
| 82 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 83 | INITIALIZE_PASS_BEGIN(LowerIntrinsics, "gc-lowering", "GC Lowering", false, |
| 84 | false) |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 85 | INITIALIZE_PASS_DEPENDENCY(GCModuleInfo) |
| 86 | INITIALIZE_PASS_END(LowerIntrinsics, "gc-lowering", "GC Lowering", false, false) |
| 87 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 88 | FunctionPass *llvm::createGCLoweringPass() { return new LowerIntrinsics(); } |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 89 | |
| 90 | char LowerIntrinsics::ID = 0; |
| 91 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 92 | LowerIntrinsics::LowerIntrinsics() : FunctionPass(ID) { |
| 93 | initializeLowerIntrinsicsPass(*PassRegistry::getPassRegistry()); |
| 94 | } |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 95 | |
| 96 | const char *LowerIntrinsics::getPassName() const { |
| 97 | return "Lower Garbage Collection Instructions"; |
| 98 | } |
| 99 | |
| 100 | void LowerIntrinsics::getAnalysisUsage(AnalysisUsage &AU) const { |
| 101 | FunctionPass::getAnalysisUsage(AU); |
| 102 | AU.addRequired<GCModuleInfo>(); |
| 103 | AU.addPreserved<DominatorTreeWrapperPass>(); |
| 104 | } |
| 105 | |
Philip Reames | 66c9fb0 | 2015-01-15 19:49:25 +0000 | [diff] [blame] | 106 | static bool NeedsDefaultLoweringPass(const GCStrategy &C) { |
| 107 | // Default lowering is necessary only if read or write barriers have a default |
| 108 | // action. The default for roots is no action. |
| 109 | return !C.customWriteBarrier() || !C.customReadBarrier() || |
| 110 | C.initializeRoots(); |
| 111 | } |
| 112 | |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 113 | /// doInitialization - If this module uses the GC intrinsics, find them now. |
| 114 | bool LowerIntrinsics::doInitialization(Module &M) { |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 115 | GCModuleInfo *MI = getAnalysisIfAvailable<GCModuleInfo>(); |
| 116 | assert(MI && "LowerIntrinsics didn't require GCModuleInfo!?"); |
| 117 | for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) |
| 118 | if (!I->isDeclaration() && I->hasGC()) |
| 119 | MI->getFunctionInfo(*I); // Instantiate the GC strategy. |
| 120 | |
Philip Reames | 23cf2e2 | 2015-01-28 19:28:03 +0000 | [diff] [blame] | 121 | return false; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 122 | } |
| 123 | |
Philip Reames | 66c9fb0 | 2015-01-15 19:49:25 +0000 | [diff] [blame] | 124 | /// CouldBecomeSafePoint - Predicate to conservatively determine whether the |
| 125 | /// instruction could introduce a safe point. |
| 126 | static bool CouldBecomeSafePoint(Instruction *I) { |
| 127 | // The natural definition of instructions which could introduce safe points |
| 128 | // are: |
| 129 | // |
| 130 | // - call, invoke (AfterCall, BeforeCall) |
| 131 | // - phis (Loops) |
| 132 | // - invoke, ret, unwind (Exit) |
| 133 | // |
| 134 | // However, instructions as seemingly inoccuous as arithmetic can become |
| 135 | // libcalls upon lowering (e.g., div i64 on a 32-bit platform), so instead |
| 136 | // it is necessary to take a conservative approach. |
| 137 | |
| 138 | if (isa<AllocaInst>(I) || isa<GetElementPtrInst>(I) || isa<StoreInst>(I) || |
| 139 | isa<LoadInst>(I)) |
| 140 | return false; |
| 141 | |
| 142 | // llvm.gcroot is safe because it doesn't do anything at runtime. |
| 143 | if (CallInst *CI = dyn_cast<CallInst>(I)) |
| 144 | if (Function *F = CI->getCalledFunction()) |
| 145 | if (unsigned IID = F->getIntrinsicID()) |
| 146 | if (IID == Intrinsic::gcroot) |
| 147 | return false; |
| 148 | |
| 149 | return true; |
| 150 | } |
| 151 | |
| 152 | static bool InsertRootInitializers(Function &F, AllocaInst **Roots, |
| 153 | unsigned Count) { |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 154 | // Scroll past alloca instructions. |
| 155 | BasicBlock::iterator IP = F.getEntryBlock().begin(); |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 156 | while (isa<AllocaInst>(IP)) |
| 157 | ++IP; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 158 | |
| 159 | // Search for initializers in the initial BB. |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 160 | SmallPtrSet<AllocaInst *, 16> InitedRoots; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 161 | for (; !CouldBecomeSafePoint(IP); ++IP) |
| 162 | if (StoreInst *SI = dyn_cast<StoreInst>(IP)) |
| 163 | if (AllocaInst *AI = |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 164 | dyn_cast<AllocaInst>(SI->getOperand(1)->stripPointerCasts())) |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 165 | InitedRoots.insert(AI); |
| 166 | |
| 167 | // Add root initializers. |
| 168 | bool MadeChange = false; |
| 169 | |
| 170 | for (AllocaInst **I = Roots, **E = Roots + Count; I != E; ++I) |
| 171 | if (!InitedRoots.count(*I)) { |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 172 | StoreInst *SI = new StoreInst( |
| 173 | ConstantPointerNull::get(cast<PointerType>( |
| 174 | cast<PointerType>((*I)->getType())->getElementType())), |
| 175 | *I); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 176 | SI->insertAfter(*I); |
| 177 | MadeChange = true; |
| 178 | } |
| 179 | |
| 180 | return MadeChange; |
| 181 | } |
| 182 | |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 183 | /// runOnFunction - Replace gcread/gcwrite intrinsics with loads and stores. |
| 184 | /// Leave gcroot intrinsics; the code generator needs to see those. |
| 185 | bool LowerIntrinsics::runOnFunction(Function &F) { |
| 186 | // Quick exit for functions that do not use GC. |
| 187 | if (!F.hasGC()) |
| 188 | return false; |
| 189 | |
| 190 | GCFunctionInfo &FI = getAnalysis<GCModuleInfo>().getFunctionInfo(F); |
| 191 | GCStrategy &S = FI.getStrategy(); |
| 192 | |
| 193 | bool MadeChange = false; |
| 194 | |
| 195 | if (NeedsDefaultLoweringPass(S)) |
| 196 | MadeChange |= PerformDefaultLowering(F, S); |
| 197 | |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 198 | return MadeChange; |
| 199 | } |
| 200 | |
| 201 | bool LowerIntrinsics::PerformDefaultLowering(Function &F, GCStrategy &S) { |
| 202 | bool LowerWr = !S.customWriteBarrier(); |
| 203 | bool LowerRd = !S.customReadBarrier(); |
| 204 | bool InitRoots = S.initializeRoots(); |
| 205 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 206 | SmallVector<AllocaInst *, 32> Roots; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 207 | |
| 208 | bool MadeChange = false; |
| 209 | for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) { |
| 210 | for (BasicBlock::iterator II = BB->begin(), E = BB->end(); II != E;) { |
| 211 | if (IntrinsicInst *CI = dyn_cast<IntrinsicInst>(II++)) { |
| 212 | Function *F = CI->getCalledFunction(); |
| 213 | switch (F->getIntrinsicID()) { |
| 214 | case Intrinsic::gcwrite: |
| 215 | if (LowerWr) { |
| 216 | // Replace a write barrier with a simple store. |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 217 | Value *St = |
| 218 | new StoreInst(CI->getArgOperand(0), CI->getArgOperand(2), CI); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 219 | CI->replaceAllUsesWith(St); |
| 220 | CI->eraseFromParent(); |
| 221 | } |
| 222 | break; |
| 223 | case Intrinsic::gcread: |
| 224 | if (LowerRd) { |
| 225 | // Replace a read barrier with a simple load. |
| 226 | Value *Ld = new LoadInst(CI->getArgOperand(1), "", CI); |
| 227 | Ld->takeName(CI); |
| 228 | CI->replaceAllUsesWith(Ld); |
| 229 | CI->eraseFromParent(); |
| 230 | } |
| 231 | break; |
| 232 | case Intrinsic::gcroot: |
| 233 | if (InitRoots) { |
| 234 | // Initialize the GC root, but do not delete the intrinsic. The |
| 235 | // backend needs the intrinsic to flag the stack slot. |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 236 | Roots.push_back( |
| 237 | cast<AllocaInst>(CI->getArgOperand(0)->stripPointerCasts())); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 238 | } |
| 239 | break; |
| 240 | default: |
| 241 | continue; |
| 242 | } |
| 243 | |
| 244 | MadeChange = true; |
| 245 | } |
| 246 | } |
| 247 | } |
| 248 | |
| 249 | if (Roots.size()) |
| 250 | MadeChange |= InsertRootInitializers(F, Roots.begin(), Roots.size()); |
| 251 | |
| 252 | return MadeChange; |
| 253 | } |
| 254 | |
| 255 | // ----------------------------------------------------------------------------- |
| 256 | |
| 257 | char GCMachineCodeAnalysis::ID = 0; |
| 258 | char &llvm::GCMachineCodeAnalysisID = GCMachineCodeAnalysis::ID; |
| 259 | |
| 260 | INITIALIZE_PASS(GCMachineCodeAnalysis, "gc-analysis", |
| 261 | "Analyze Machine Code For Garbage Collection", false, false) |
| 262 | |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 263 | GCMachineCodeAnalysis::GCMachineCodeAnalysis() : MachineFunctionPass(ID) {} |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 264 | |
| 265 | void GCMachineCodeAnalysis::getAnalysisUsage(AnalysisUsage &AU) const { |
| 266 | MachineFunctionPass::getAnalysisUsage(AU); |
| 267 | AU.setPreservesAll(); |
| 268 | AU.addRequired<MachineModuleInfo>(); |
| 269 | AU.addRequired<GCModuleInfo>(); |
| 270 | } |
| 271 | |
| 272 | MCSymbol *GCMachineCodeAnalysis::InsertLabel(MachineBasicBlock &MBB, |
| 273 | MachineBasicBlock::iterator MI, |
| 274 | DebugLoc DL) const { |
| 275 | MCSymbol *Label = MBB.getParent()->getContext().CreateTempSymbol(); |
| 276 | BuildMI(MBB, MI, DL, TII->get(TargetOpcode::GC_LABEL)).addSym(Label); |
| 277 | return Label; |
| 278 | } |
| 279 | |
| 280 | void GCMachineCodeAnalysis::VisitCallPoint(MachineBasicBlock::iterator CI) { |
| 281 | // Find the return address (next instruction), too, so as to bracket the call |
| 282 | // instruction. |
| 283 | MachineBasicBlock::iterator RAI = CI; |
| 284 | ++RAI; |
| 285 | |
| 286 | if (FI->getStrategy().needsSafePoint(GC::PreCall)) { |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 287 | MCSymbol *Label = InsertLabel(*CI->getParent(), CI, CI->getDebugLoc()); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 288 | FI->addSafePoint(GC::PreCall, Label, CI->getDebugLoc()); |
| 289 | } |
| 290 | |
| 291 | if (FI->getStrategy().needsSafePoint(GC::PostCall)) { |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 292 | MCSymbol *Label = InsertLabel(*CI->getParent(), RAI, CI->getDebugLoc()); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 293 | FI->addSafePoint(GC::PostCall, Label, CI->getDebugLoc()); |
| 294 | } |
| 295 | } |
| 296 | |
| 297 | void GCMachineCodeAnalysis::FindSafePoints(MachineFunction &MF) { |
Philip Reames | b871441 | 2015-01-15 19:39:17 +0000 | [diff] [blame] | 298 | for (MachineFunction::iterator BBI = MF.begin(), BBE = MF.end(); BBI != BBE; |
| 299 | ++BBI) |
| 300 | for (MachineBasicBlock::iterator MI = BBI->begin(), ME = BBI->end(); |
| 301 | MI != ME; ++MI) |
Philip Reames | 7de640a | 2015-01-16 19:33:28 +0000 | [diff] [blame] | 302 | if (MI->isCall()) { |
| 303 | // Do not treat tail or sibling call sites as safe points. This is |
| 304 | // legal since any arguments passed to the callee which live in the |
| 305 | // remnants of the callers frame will be owned and updated by the |
| 306 | // callee if required. |
| 307 | if (MI->isTerminator()) |
| 308 | continue; |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 309 | VisitCallPoint(MI); |
Philip Reames | 7de640a | 2015-01-16 19:33:28 +0000 | [diff] [blame] | 310 | } |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 311 | } |
| 312 | |
| 313 | void GCMachineCodeAnalysis::FindStackOffsets(MachineFunction &MF) { |
Eric Christopher | 1df0c51 | 2015-02-20 18:44:15 +0000 | [diff] [blame] | 314 | const TargetFrameLowering *TFI = MF.getSubtarget().getFrameLowering(); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 315 | assert(TFI && "TargetRegisterInfo not available!"); |
| 316 | |
| 317 | for (GCFunctionInfo::roots_iterator RI = FI->roots_begin(); |
| 318 | RI != FI->roots_end();) { |
| 319 | // If the root references a dead object, no need to keep it. |
| 320 | if (MF.getFrameInfo()->isDeadObjectIndex(RI->Num)) { |
| 321 | RI = FI->removeStackRoot(RI); |
| 322 | } else { |
| 323 | RI->StackOffset = TFI->getFrameIndexOffset(MF, RI->Num); |
| 324 | ++RI; |
| 325 | } |
| 326 | } |
| 327 | } |
| 328 | |
| 329 | bool GCMachineCodeAnalysis::runOnMachineFunction(MachineFunction &MF) { |
| 330 | // Quick exit for functions that do not use GC. |
| 331 | if (!MF.getFunction()->hasGC()) |
| 332 | return false; |
| 333 | |
| 334 | FI = &getAnalysis<GCModuleInfo>().getFunctionInfo(*MF.getFunction()); |
| 335 | if (!FI->getStrategy().needsSafePoints()) |
| 336 | return false; |
| 337 | |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 338 | MMI = &getAnalysis<MachineModuleInfo>(); |
Eric Christopher | 1df0c51 | 2015-02-20 18:44:15 +0000 | [diff] [blame] | 339 | TII = MF.getSubtarget().getInstrInfo(); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 340 | |
| 341 | // Find the size of the stack frame. |
| 342 | FI->setFrameSize(MF.getFrameInfo()->getStackSize()); |
| 343 | |
| 344 | // Find all safe points. |
Philip Reames | 7de640a | 2015-01-16 19:33:28 +0000 | [diff] [blame] | 345 | FindSafePoints(MF); |
Philip Reames | f27f373 | 2015-01-15 19:29:42 +0000 | [diff] [blame] | 346 | |
| 347 | // Find the stack offsets for all roots. |
| 348 | FindStackOffsets(MF); |
| 349 | |
| 350 | return false; |
| 351 | } |