diff options
author | Gabor Greif <ggreif@gmail.com> | 2010-07-23 10:23:01 +0000 |
---|---|---|
committer | Gabor Greif <ggreif@gmail.com> | 2010-07-23 10:23:01 +0000 |
commit | a1e9c98302ae861dc6c35ef80b570aec7fed0504 (patch) | |
tree | 3c6cef35e7492e4584d9650ff368fb3534f13db5 /llvm/lib/Target/PIC16 | |
parent | 849a8cf81554f8bb210778c3af24836d28665339 (diff) | |
download | bcm5719-llvm-a1e9c98302ae861dc6c35ef80b570aec7fed0504.tar.gz bcm5719-llvm-a1e9c98302ae861dc6c35ef80b570aec7fed0504.zip |
do not (implicitly) dereference iterator many times, cache it instead
llvm-svn: 109222
Diffstat (limited to 'llvm/lib/Target/PIC16')
-rw-r--r-- | llvm/lib/Target/PIC16/PIC16Passes/PIC16Overlay.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Target/PIC16/PIC16Passes/PIC16Overlay.cpp b/llvm/lib/Target/PIC16/PIC16Passes/PIC16Overlay.cpp index 5ecb6aa5515..0f8928a4b5f 100644 --- a/llvm/lib/Target/PIC16/PIC16Passes/PIC16Overlay.cpp +++ b/llvm/lib/Target/PIC16/PIC16Passes/PIC16Overlay.cpp @@ -171,8 +171,9 @@ void PIC16Overlay::MarkIndirectlyCalledFunctions(Module &M) { for (Module::iterator MI = M.begin(), E = M.end(); MI != E; ++MI) { for (Value::use_iterator I = MI->use_begin(), E = MI->use_end(); I != E; ++I) { - if ((!isa<CallInst>(I) && !isa<InvokeInst>(I)) - || !CallSite(cast<Instruction>(I)).isCallee(I)) { + User *U = *I; + if ((!isa<CallInst>(U) && !isa<InvokeInst>(U)) + || !CallSite(cast<Instruction>(U)).isCallee(I)) { setColor(MI, ++IndirectCallColor); break; } |