BU: Avoid variable shadowing, C++11-ify loops while at it.  NFC.

llvm-svn: 238380
diff --git a/poolalloc/lib/DSA/BottomUpClosure.cpp b/poolalloc/lib/DSA/BottomUpClosure.cpp
index 02e561c..98ade51 100644
--- a/poolalloc/lib/DSA/BottomUpClosure.cpp
+++ b/poolalloc/lib/DSA/BottomUpClosure.cpp
@@ -632,11 +632,9 @@
   DSGraph::FunctionListTy &AuxCallsList = Graph->getAuxFunctionCalls();
   TempFCs.swap(AuxCallsList);
 
-  for(DSGraph::FunctionListTy::iterator I = TempFCs.begin(), E = TempFCs.end();
-      I != E; ++I) {
+  for (auto &CS : TempFCs) {
     DEBUG(Graph->AssertGraphOK(); Graph->getGlobalsGraph()->AssertGraphOK());
 
-    DSCallSite &CS = *I;
 
     // Fast path for noop calls.  Note that we don't care about merging globals
     // in the callee with nodes in the caller here.
@@ -676,9 +674,7 @@
 
     DSGraph *GI;
 
-    for (FuncSet::iterator I = CalledFuncs.begin(), E = CalledFuncs.end();
-         I != E; ++I) {
-      const Function *Callee = *I;
+    for (auto *Callee : CalledFuncs) {
       // Get the data structure graph for the called function.
 
       GI = getDSGraph(*Callee);  // Graph to inline