diff options
author | Gabor Greif <ggreif@gmail.com> | 2010-07-09 13:17:13 +0000 |
---|---|---|
committer | Gabor Greif <ggreif@gmail.com> | 2010-07-09 13:17:13 +0000 |
commit | 517e124d53a673c35e34feec39304a1c6448d371 (patch) | |
tree | 03030bf4bd9a0a4b443f195cb77824e57b1ffebd /lib/Analysis/IPA | |
parent | b654435712f3e1345cc40aff99724de9300e5de0 (diff) | |
download | external_llvm-517e124d53a673c35e34feec39304a1c6448d371.zip external_llvm-517e124d53a673c35e34feec39304a1c6448d371.tar.gz external_llvm-517e124d53a673c35e34feec39304a1c6448d371.tar.bz2 |
do not repeatedly dereference use_iterator
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@107963 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Analysis/IPA')
-rw-r--r-- | lib/Analysis/IPA/CallGraph.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/Analysis/IPA/CallGraph.cpp b/lib/Analysis/IPA/CallGraph.cpp index 2bde56d..65c7c6e 100644 --- a/lib/Analysis/IPA/CallGraph.cpp +++ b/lib/Analysis/IPA/CallGraph.cpp @@ -126,13 +126,15 @@ private: } // Loop over all of the users of the function, looking for non-call uses. - for (Value::use_iterator I = F->use_begin(), E = F->use_end(); I != E; ++I) - if ((!isa<CallInst>(I) && !isa<InvokeInst>(I)) - || !CallSite(cast<Instruction>(I)).isCallee(I)) { + for (Value::use_iterator I = F->use_begin(), E = F->use_end(); I != E; ++I){ + User *U = *I; + if ((!isa<CallInst>(U) && !isa<InvokeInst>(U)) + || !CallSite(cast<Instruction>(U)).isCallee(I)) { // Not a call, or being used as a parameter rather than as the callee. ExternalCallingNode->addCalledFunction(CallSite(), Node); break; } + } // If this function is not defined in this translation unit, it could call // anything. |