From 3949b9e6dd1bfc7f6f83b281afa2a49fa91a61b8 Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Wed, 13 Apr 2016 05:25:12 +0000 Subject: [PATCH] Modernize Internalizer with for-range loop (NFC) From: Mehdi Amini llvm-svn: 266168 --- llvm/lib/Transforms/IPO/Internalize.cpp | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/llvm/lib/Transforms/IPO/Internalize.cpp b/llvm/lib/Transforms/IPO/Internalize.cpp index eac4c7b..f438b4b 100644 --- a/llvm/lib/Transforms/IPO/Internalize.cpp +++ b/llvm/lib/Transforms/IPO/Internalize.cpp @@ -275,23 +275,21 @@ bool Internalizer::internalizeModule(Module &M, CallGraph *CG) { // Mark all global variables with initializers that are not in the api as // internal as well. - for (Module::global_iterator I = M.global_begin(), E = M.global_end(); I != E; - ++I) { - if (!maybeInternalize(*I, ExternalComdats)) + for (auto &GV : M.globals()) { + if (!maybeInternalize(GV, ExternalComdats)) continue; ++NumGlobals; - DEBUG(dbgs() << "Internalized gvar " << I->getName() << "\n"); + DEBUG(dbgs() << "Internalized gvar " << GV.getName() << "\n"); } // Mark all aliases that are not in the api as internal as well. - for (Module::alias_iterator I = M.alias_begin(), E = M.alias_end(); I != E; - ++I) { - if (!maybeInternalize(*I, ExternalComdats)) + for (auto &GA : M.aliases()) { + if (!maybeInternalize(GA, ExternalComdats)) continue; ++NumAliases; - DEBUG(dbgs() << "Internalized alias " << I->getName() << "\n"); + DEBUG(dbgs() << "Internalized alias " << GA.getName() << "\n"); } // We do not keep track of whether this pass changed the module because -- 2.7.4