xref: /aosp_15_r20/external/llvm/lib/Transforms/ObjCARC/DependencyAnalysis.cpp (revision 9880d6810fe72a1726cb53787c6711e909410d58)
1*9880d681SAndroid Build Coastguard Worker //===- DependencyAnalysis.cpp - ObjC ARC Optimization ---------------------===//
2*9880d681SAndroid Build Coastguard Worker //
3*9880d681SAndroid Build Coastguard Worker //                     The LLVM Compiler Infrastructure
4*9880d681SAndroid Build Coastguard Worker //
5*9880d681SAndroid Build Coastguard Worker // This file is distributed under the University of Illinois Open Source
6*9880d681SAndroid Build Coastguard Worker // License. See LICENSE.TXT for details.
7*9880d681SAndroid Build Coastguard Worker //
8*9880d681SAndroid Build Coastguard Worker //===----------------------------------------------------------------------===//
9*9880d681SAndroid Build Coastguard Worker /// \file
10*9880d681SAndroid Build Coastguard Worker ///
11*9880d681SAndroid Build Coastguard Worker /// This file defines special dependency analysis routines used in Objective C
12*9880d681SAndroid Build Coastguard Worker /// ARC Optimizations.
13*9880d681SAndroid Build Coastguard Worker ///
14*9880d681SAndroid Build Coastguard Worker /// WARNING: This file knows about certain library functions. It recognizes them
15*9880d681SAndroid Build Coastguard Worker /// by name, and hardwires knowledge of their semantics.
16*9880d681SAndroid Build Coastguard Worker ///
17*9880d681SAndroid Build Coastguard Worker /// WARNING: This file knows about how certain Objective-C library functions are
18*9880d681SAndroid Build Coastguard Worker /// used. Naive LLVM IR transformations which would otherwise be
19*9880d681SAndroid Build Coastguard Worker /// behavior-preserving may break these assumptions.
20*9880d681SAndroid Build Coastguard Worker ///
21*9880d681SAndroid Build Coastguard Worker //===----------------------------------------------------------------------===//
22*9880d681SAndroid Build Coastguard Worker 
23*9880d681SAndroid Build Coastguard Worker #include "ObjCARC.h"
24*9880d681SAndroid Build Coastguard Worker #include "DependencyAnalysis.h"
25*9880d681SAndroid Build Coastguard Worker #include "ProvenanceAnalysis.h"
26*9880d681SAndroid Build Coastguard Worker #include "llvm/IR/CFG.h"
27*9880d681SAndroid Build Coastguard Worker 
28*9880d681SAndroid Build Coastguard Worker using namespace llvm;
29*9880d681SAndroid Build Coastguard Worker using namespace llvm::objcarc;
30*9880d681SAndroid Build Coastguard Worker 
31*9880d681SAndroid Build Coastguard Worker #define DEBUG_TYPE "objc-arc-dependency"
32*9880d681SAndroid Build Coastguard Worker 
33*9880d681SAndroid Build Coastguard Worker /// Test whether the given instruction can result in a reference count
34*9880d681SAndroid Build Coastguard Worker /// modification (positive or negative) for the pointer's object.
CanAlterRefCount(const Instruction * Inst,const Value * Ptr,ProvenanceAnalysis & PA,ARCInstKind Class)35*9880d681SAndroid Build Coastguard Worker bool llvm::objcarc::CanAlterRefCount(const Instruction *Inst, const Value *Ptr,
36*9880d681SAndroid Build Coastguard Worker                                      ProvenanceAnalysis &PA,
37*9880d681SAndroid Build Coastguard Worker                                      ARCInstKind Class) {
38*9880d681SAndroid Build Coastguard Worker   switch (Class) {
39*9880d681SAndroid Build Coastguard Worker   case ARCInstKind::Autorelease:
40*9880d681SAndroid Build Coastguard Worker   case ARCInstKind::AutoreleaseRV:
41*9880d681SAndroid Build Coastguard Worker   case ARCInstKind::IntrinsicUser:
42*9880d681SAndroid Build Coastguard Worker   case ARCInstKind::User:
43*9880d681SAndroid Build Coastguard Worker     // These operations never directly modify a reference count.
44*9880d681SAndroid Build Coastguard Worker     return false;
45*9880d681SAndroid Build Coastguard Worker   default: break;
46*9880d681SAndroid Build Coastguard Worker   }
47*9880d681SAndroid Build Coastguard Worker 
48*9880d681SAndroid Build Coastguard Worker   ImmutableCallSite CS(Inst);
49*9880d681SAndroid Build Coastguard Worker   assert(CS && "Only calls can alter reference counts!");
50*9880d681SAndroid Build Coastguard Worker 
51*9880d681SAndroid Build Coastguard Worker   // See if AliasAnalysis can help us with the call.
52*9880d681SAndroid Build Coastguard Worker   FunctionModRefBehavior MRB = PA.getAA()->getModRefBehavior(CS);
53*9880d681SAndroid Build Coastguard Worker   if (AliasAnalysis::onlyReadsMemory(MRB))
54*9880d681SAndroid Build Coastguard Worker     return false;
55*9880d681SAndroid Build Coastguard Worker   if (AliasAnalysis::onlyAccessesArgPointees(MRB)) {
56*9880d681SAndroid Build Coastguard Worker     const DataLayout &DL = Inst->getModule()->getDataLayout();
57*9880d681SAndroid Build Coastguard Worker     for (ImmutableCallSite::arg_iterator I = CS.arg_begin(), E = CS.arg_end();
58*9880d681SAndroid Build Coastguard Worker          I != E; ++I) {
59*9880d681SAndroid Build Coastguard Worker       const Value *Op = *I;
60*9880d681SAndroid Build Coastguard Worker       if (IsPotentialRetainableObjPtr(Op, *PA.getAA()) &&
61*9880d681SAndroid Build Coastguard Worker           PA.related(Ptr, Op, DL))
62*9880d681SAndroid Build Coastguard Worker         return true;
63*9880d681SAndroid Build Coastguard Worker     }
64*9880d681SAndroid Build Coastguard Worker     return false;
65*9880d681SAndroid Build Coastguard Worker   }
66*9880d681SAndroid Build Coastguard Worker 
67*9880d681SAndroid Build Coastguard Worker   // Assume the worst.
68*9880d681SAndroid Build Coastguard Worker   return true;
69*9880d681SAndroid Build Coastguard Worker }
70*9880d681SAndroid Build Coastguard Worker 
CanDecrementRefCount(const Instruction * Inst,const Value * Ptr,ProvenanceAnalysis & PA,ARCInstKind Class)71*9880d681SAndroid Build Coastguard Worker bool llvm::objcarc::CanDecrementRefCount(const Instruction *Inst,
72*9880d681SAndroid Build Coastguard Worker                                          const Value *Ptr,
73*9880d681SAndroid Build Coastguard Worker                                          ProvenanceAnalysis &PA,
74*9880d681SAndroid Build Coastguard Worker                                          ARCInstKind Class) {
75*9880d681SAndroid Build Coastguard Worker   // First perform a quick check if Class can not touch ref counts.
76*9880d681SAndroid Build Coastguard Worker   if (!CanDecrementRefCount(Class))
77*9880d681SAndroid Build Coastguard Worker     return false;
78*9880d681SAndroid Build Coastguard Worker 
79*9880d681SAndroid Build Coastguard Worker   // Otherwise, just use CanAlterRefCount for now.
80*9880d681SAndroid Build Coastguard Worker   return CanAlterRefCount(Inst, Ptr, PA, Class);
81*9880d681SAndroid Build Coastguard Worker }
82*9880d681SAndroid Build Coastguard Worker 
83*9880d681SAndroid Build Coastguard Worker /// Test whether the given instruction can "use" the given pointer's object in a
84*9880d681SAndroid Build Coastguard Worker /// way that requires the reference count to be positive.
CanUse(const Instruction * Inst,const Value * Ptr,ProvenanceAnalysis & PA,ARCInstKind Class)85*9880d681SAndroid Build Coastguard Worker bool llvm::objcarc::CanUse(const Instruction *Inst, const Value *Ptr,
86*9880d681SAndroid Build Coastguard Worker                            ProvenanceAnalysis &PA, ARCInstKind Class) {
87*9880d681SAndroid Build Coastguard Worker   // ARCInstKind::Call operations (as opposed to
88*9880d681SAndroid Build Coastguard Worker   // ARCInstKind::CallOrUser) never "use" objc pointers.
89*9880d681SAndroid Build Coastguard Worker   if (Class == ARCInstKind::Call)
90*9880d681SAndroid Build Coastguard Worker     return false;
91*9880d681SAndroid Build Coastguard Worker 
92*9880d681SAndroid Build Coastguard Worker   const DataLayout &DL = Inst->getModule()->getDataLayout();
93*9880d681SAndroid Build Coastguard Worker 
94*9880d681SAndroid Build Coastguard Worker   // Consider various instructions which may have pointer arguments which are
95*9880d681SAndroid Build Coastguard Worker   // not "uses".
96*9880d681SAndroid Build Coastguard Worker   if (const ICmpInst *ICI = dyn_cast<ICmpInst>(Inst)) {
97*9880d681SAndroid Build Coastguard Worker     // Comparing a pointer with null, or any other constant, isn't really a use,
98*9880d681SAndroid Build Coastguard Worker     // because we don't care what the pointer points to, or about the values
99*9880d681SAndroid Build Coastguard Worker     // of any other dynamic reference-counted pointers.
100*9880d681SAndroid Build Coastguard Worker     if (!IsPotentialRetainableObjPtr(ICI->getOperand(1), *PA.getAA()))
101*9880d681SAndroid Build Coastguard Worker       return false;
102*9880d681SAndroid Build Coastguard Worker   } else if (auto CS = ImmutableCallSite(Inst)) {
103*9880d681SAndroid Build Coastguard Worker     // For calls, just check the arguments (and not the callee operand).
104*9880d681SAndroid Build Coastguard Worker     for (ImmutableCallSite::arg_iterator OI = CS.arg_begin(),
105*9880d681SAndroid Build Coastguard Worker          OE = CS.arg_end(); OI != OE; ++OI) {
106*9880d681SAndroid Build Coastguard Worker       const Value *Op = *OI;
107*9880d681SAndroid Build Coastguard Worker       if (IsPotentialRetainableObjPtr(Op, *PA.getAA()) &&
108*9880d681SAndroid Build Coastguard Worker           PA.related(Ptr, Op, DL))
109*9880d681SAndroid Build Coastguard Worker         return true;
110*9880d681SAndroid Build Coastguard Worker     }
111*9880d681SAndroid Build Coastguard Worker     return false;
112*9880d681SAndroid Build Coastguard Worker   } else if (const StoreInst *SI = dyn_cast<StoreInst>(Inst)) {
113*9880d681SAndroid Build Coastguard Worker     // Special-case stores, because we don't care about the stored value, just
114*9880d681SAndroid Build Coastguard Worker     // the store address.
115*9880d681SAndroid Build Coastguard Worker     const Value *Op = GetUnderlyingObjCPtr(SI->getPointerOperand(), DL);
116*9880d681SAndroid Build Coastguard Worker     // If we can't tell what the underlying object was, assume there is a
117*9880d681SAndroid Build Coastguard Worker     // dependence.
118*9880d681SAndroid Build Coastguard Worker     return IsPotentialRetainableObjPtr(Op, *PA.getAA()) &&
119*9880d681SAndroid Build Coastguard Worker            PA.related(Op, Ptr, DL);
120*9880d681SAndroid Build Coastguard Worker   }
121*9880d681SAndroid Build Coastguard Worker 
122*9880d681SAndroid Build Coastguard Worker   // Check each operand for a match.
123*9880d681SAndroid Build Coastguard Worker   for (User::const_op_iterator OI = Inst->op_begin(), OE = Inst->op_end();
124*9880d681SAndroid Build Coastguard Worker        OI != OE; ++OI) {
125*9880d681SAndroid Build Coastguard Worker     const Value *Op = *OI;
126*9880d681SAndroid Build Coastguard Worker     if (IsPotentialRetainableObjPtr(Op, *PA.getAA()) && PA.related(Ptr, Op, DL))
127*9880d681SAndroid Build Coastguard Worker       return true;
128*9880d681SAndroid Build Coastguard Worker   }
129*9880d681SAndroid Build Coastguard Worker   return false;
130*9880d681SAndroid Build Coastguard Worker }
131*9880d681SAndroid Build Coastguard Worker 
132*9880d681SAndroid Build Coastguard Worker /// Test if there can be dependencies on Inst through Arg. This function only
133*9880d681SAndroid Build Coastguard Worker /// tests dependencies relevant for removing pairs of calls.
134*9880d681SAndroid Build Coastguard Worker bool
Depends(DependenceKind Flavor,Instruction * Inst,const Value * Arg,ProvenanceAnalysis & PA)135*9880d681SAndroid Build Coastguard Worker llvm::objcarc::Depends(DependenceKind Flavor, Instruction *Inst,
136*9880d681SAndroid Build Coastguard Worker                        const Value *Arg, ProvenanceAnalysis &PA) {
137*9880d681SAndroid Build Coastguard Worker   // If we've reached the definition of Arg, stop.
138*9880d681SAndroid Build Coastguard Worker   if (Inst == Arg)
139*9880d681SAndroid Build Coastguard Worker     return true;
140*9880d681SAndroid Build Coastguard Worker 
141*9880d681SAndroid Build Coastguard Worker   switch (Flavor) {
142*9880d681SAndroid Build Coastguard Worker   case NeedsPositiveRetainCount: {
143*9880d681SAndroid Build Coastguard Worker     ARCInstKind Class = GetARCInstKind(Inst);
144*9880d681SAndroid Build Coastguard Worker     switch (Class) {
145*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPop:
146*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPush:
147*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::None:
148*9880d681SAndroid Build Coastguard Worker       return false;
149*9880d681SAndroid Build Coastguard Worker     default:
150*9880d681SAndroid Build Coastguard Worker       return CanUse(Inst, Arg, PA, Class);
151*9880d681SAndroid Build Coastguard Worker     }
152*9880d681SAndroid Build Coastguard Worker   }
153*9880d681SAndroid Build Coastguard Worker 
154*9880d681SAndroid Build Coastguard Worker   case AutoreleasePoolBoundary: {
155*9880d681SAndroid Build Coastguard Worker     ARCInstKind Class = GetARCInstKind(Inst);
156*9880d681SAndroid Build Coastguard Worker     switch (Class) {
157*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPop:
158*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPush:
159*9880d681SAndroid Build Coastguard Worker       // These mark the end and begin of an autorelease pool scope.
160*9880d681SAndroid Build Coastguard Worker       return true;
161*9880d681SAndroid Build Coastguard Worker     default:
162*9880d681SAndroid Build Coastguard Worker       // Nothing else does this.
163*9880d681SAndroid Build Coastguard Worker       return false;
164*9880d681SAndroid Build Coastguard Worker     }
165*9880d681SAndroid Build Coastguard Worker   }
166*9880d681SAndroid Build Coastguard Worker 
167*9880d681SAndroid Build Coastguard Worker   case CanChangeRetainCount: {
168*9880d681SAndroid Build Coastguard Worker     ARCInstKind Class = GetARCInstKind(Inst);
169*9880d681SAndroid Build Coastguard Worker     switch (Class) {
170*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPop:
171*9880d681SAndroid Build Coastguard Worker       // Conservatively assume this can decrement any count.
172*9880d681SAndroid Build Coastguard Worker       return true;
173*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPush:
174*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::None:
175*9880d681SAndroid Build Coastguard Worker       return false;
176*9880d681SAndroid Build Coastguard Worker     default:
177*9880d681SAndroid Build Coastguard Worker       return CanAlterRefCount(Inst, Arg, PA, Class);
178*9880d681SAndroid Build Coastguard Worker     }
179*9880d681SAndroid Build Coastguard Worker   }
180*9880d681SAndroid Build Coastguard Worker 
181*9880d681SAndroid Build Coastguard Worker   case RetainAutoreleaseDep:
182*9880d681SAndroid Build Coastguard Worker     switch (GetBasicARCInstKind(Inst)) {
183*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPop:
184*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::AutoreleasepoolPush:
185*9880d681SAndroid Build Coastguard Worker       // Don't merge an objc_autorelease with an objc_retain inside a different
186*9880d681SAndroid Build Coastguard Worker       // autoreleasepool scope.
187*9880d681SAndroid Build Coastguard Worker       return true;
188*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::Retain:
189*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::RetainRV:
190*9880d681SAndroid Build Coastguard Worker       // Check for a retain of the same pointer for merging.
191*9880d681SAndroid Build Coastguard Worker       return GetArgRCIdentityRoot(Inst) == Arg;
192*9880d681SAndroid Build Coastguard Worker     default:
193*9880d681SAndroid Build Coastguard Worker       // Nothing else matters for objc_retainAutorelease formation.
194*9880d681SAndroid Build Coastguard Worker       return false;
195*9880d681SAndroid Build Coastguard Worker     }
196*9880d681SAndroid Build Coastguard Worker 
197*9880d681SAndroid Build Coastguard Worker   case RetainAutoreleaseRVDep: {
198*9880d681SAndroid Build Coastguard Worker     ARCInstKind Class = GetBasicARCInstKind(Inst);
199*9880d681SAndroid Build Coastguard Worker     switch (Class) {
200*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::Retain:
201*9880d681SAndroid Build Coastguard Worker     case ARCInstKind::RetainRV:
202*9880d681SAndroid Build Coastguard Worker       // Check for a retain of the same pointer for merging.
203*9880d681SAndroid Build Coastguard Worker       return GetArgRCIdentityRoot(Inst) == Arg;
204*9880d681SAndroid Build Coastguard Worker     default:
205*9880d681SAndroid Build Coastguard Worker       // Anything that can autorelease interrupts
206*9880d681SAndroid Build Coastguard Worker       // retainAutoreleaseReturnValue formation.
207*9880d681SAndroid Build Coastguard Worker       return CanInterruptRV(Class);
208*9880d681SAndroid Build Coastguard Worker     }
209*9880d681SAndroid Build Coastguard Worker   }
210*9880d681SAndroid Build Coastguard Worker 
211*9880d681SAndroid Build Coastguard Worker   case RetainRVDep:
212*9880d681SAndroid Build Coastguard Worker     return CanInterruptRV(GetBasicARCInstKind(Inst));
213*9880d681SAndroid Build Coastguard Worker   }
214*9880d681SAndroid Build Coastguard Worker 
215*9880d681SAndroid Build Coastguard Worker   llvm_unreachable("Invalid dependence flavor");
216*9880d681SAndroid Build Coastguard Worker }
217*9880d681SAndroid Build Coastguard Worker 
218*9880d681SAndroid Build Coastguard Worker /// Walk up the CFG from StartPos (which is in StartBB) and find local and
219*9880d681SAndroid Build Coastguard Worker /// non-local dependencies on Arg.
220*9880d681SAndroid Build Coastguard Worker ///
221*9880d681SAndroid Build Coastguard Worker /// TODO: Cache results?
222*9880d681SAndroid Build Coastguard Worker void
FindDependencies(DependenceKind Flavor,const Value * Arg,BasicBlock * StartBB,Instruction * StartInst,SmallPtrSetImpl<Instruction * > & DependingInsts,SmallPtrSetImpl<const BasicBlock * > & Visited,ProvenanceAnalysis & PA)223*9880d681SAndroid Build Coastguard Worker llvm::objcarc::FindDependencies(DependenceKind Flavor,
224*9880d681SAndroid Build Coastguard Worker                                 const Value *Arg,
225*9880d681SAndroid Build Coastguard Worker                                 BasicBlock *StartBB, Instruction *StartInst,
226*9880d681SAndroid Build Coastguard Worker                                 SmallPtrSetImpl<Instruction *> &DependingInsts,
227*9880d681SAndroid Build Coastguard Worker                                 SmallPtrSetImpl<const BasicBlock *> &Visited,
228*9880d681SAndroid Build Coastguard Worker                                 ProvenanceAnalysis &PA) {
229*9880d681SAndroid Build Coastguard Worker   BasicBlock::iterator StartPos = StartInst->getIterator();
230*9880d681SAndroid Build Coastguard Worker 
231*9880d681SAndroid Build Coastguard Worker   SmallVector<std::pair<BasicBlock *, BasicBlock::iterator>, 4> Worklist;
232*9880d681SAndroid Build Coastguard Worker   Worklist.push_back(std::make_pair(StartBB, StartPos));
233*9880d681SAndroid Build Coastguard Worker   do {
234*9880d681SAndroid Build Coastguard Worker     std::pair<BasicBlock *, BasicBlock::iterator> Pair =
235*9880d681SAndroid Build Coastguard Worker       Worklist.pop_back_val();
236*9880d681SAndroid Build Coastguard Worker     BasicBlock *LocalStartBB = Pair.first;
237*9880d681SAndroid Build Coastguard Worker     BasicBlock::iterator LocalStartPos = Pair.second;
238*9880d681SAndroid Build Coastguard Worker     BasicBlock::iterator StartBBBegin = LocalStartBB->begin();
239*9880d681SAndroid Build Coastguard Worker     for (;;) {
240*9880d681SAndroid Build Coastguard Worker       if (LocalStartPos == StartBBBegin) {
241*9880d681SAndroid Build Coastguard Worker         pred_iterator PI(LocalStartBB), PE(LocalStartBB, false);
242*9880d681SAndroid Build Coastguard Worker         if (PI == PE)
243*9880d681SAndroid Build Coastguard Worker           // If we've reached the function entry, produce a null dependence.
244*9880d681SAndroid Build Coastguard Worker           DependingInsts.insert(nullptr);
245*9880d681SAndroid Build Coastguard Worker         else
246*9880d681SAndroid Build Coastguard Worker           // Add the predecessors to the worklist.
247*9880d681SAndroid Build Coastguard Worker           do {
248*9880d681SAndroid Build Coastguard Worker             BasicBlock *PredBB = *PI;
249*9880d681SAndroid Build Coastguard Worker             if (Visited.insert(PredBB).second)
250*9880d681SAndroid Build Coastguard Worker               Worklist.push_back(std::make_pair(PredBB, PredBB->end()));
251*9880d681SAndroid Build Coastguard Worker           } while (++PI != PE);
252*9880d681SAndroid Build Coastguard Worker         break;
253*9880d681SAndroid Build Coastguard Worker       }
254*9880d681SAndroid Build Coastguard Worker 
255*9880d681SAndroid Build Coastguard Worker       Instruction *Inst = &*--LocalStartPos;
256*9880d681SAndroid Build Coastguard Worker       if (Depends(Flavor, Inst, Arg, PA)) {
257*9880d681SAndroid Build Coastguard Worker         DependingInsts.insert(Inst);
258*9880d681SAndroid Build Coastguard Worker         break;
259*9880d681SAndroid Build Coastguard Worker       }
260*9880d681SAndroid Build Coastguard Worker     }
261*9880d681SAndroid Build Coastguard Worker   } while (!Worklist.empty());
262*9880d681SAndroid Build Coastguard Worker 
263*9880d681SAndroid Build Coastguard Worker   // Determine whether the original StartBB post-dominates all of the blocks we
264*9880d681SAndroid Build Coastguard Worker   // visited. If not, insert a sentinal indicating that most optimizations are
265*9880d681SAndroid Build Coastguard Worker   // not safe.
266*9880d681SAndroid Build Coastguard Worker   for (const BasicBlock *BB : Visited) {
267*9880d681SAndroid Build Coastguard Worker     if (BB == StartBB)
268*9880d681SAndroid Build Coastguard Worker       continue;
269*9880d681SAndroid Build Coastguard Worker     const TerminatorInst *TI = cast<TerminatorInst>(&BB->back());
270*9880d681SAndroid Build Coastguard Worker     for (succ_const_iterator SI(TI), SE(TI, false); SI != SE; ++SI) {
271*9880d681SAndroid Build Coastguard Worker       const BasicBlock *Succ = *SI;
272*9880d681SAndroid Build Coastguard Worker       if (Succ != StartBB && !Visited.count(Succ)) {
273*9880d681SAndroid Build Coastguard Worker         DependingInsts.insert(reinterpret_cast<Instruction *>(-1));
274*9880d681SAndroid Build Coastguard Worker         return;
275*9880d681SAndroid Build Coastguard Worker       }
276*9880d681SAndroid Build Coastguard Worker     }
277*9880d681SAndroid Build Coastguard Worker   }
278*9880d681SAndroid Build Coastguard Worker }
279