<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 17, 2014 at 10:06 AM, John Thompson <span dir="ltr"><<a href="mailto:John.Thompson.JTSoftware@gmail.com" target="_blank">John.Thompson.JTSoftware@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: jtsoftware<br>
Date: Thu Apr 17 12:06:13 2014<br>
New Revision: 206474<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=206474&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=206474&view=rev</a><br>
Log:<br>
Revised per Dmitri's comments. My first exposure to range-based for loops, thanks!<br>
<br>
Modified:<br>
    cfe/trunk/lib/Serialization/GlobalModuleIndex.cpp<br>
<br>
Modified: cfe/trunk/lib/Serialization/GlobalModuleIndex.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Serialization/GlobalModuleIndex.cpp?rev=206474&r1=206473&r2=206474&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Serialization/GlobalModuleIndex.cpp?rev=206474&r1=206473&r2=206474&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/lib/Serialization/GlobalModuleIndex.cpp (original)<br>
+++ cfe/trunk/lib/Serialization/GlobalModuleIndex.cpp Thu Apr 17 12:06:13 2014<br>
@@ -350,18 +350,16 @@ void GlobalModuleIndex::printStats() {<br>
 }<br>
<br>
 void GlobalModuleIndex::dump() {<br>
-  std::fprintf(stderr, "*** Global Module Index Dump:\n");<br>
-  std::fprintf(stderr, "Module files:\n");<br>
-  for (llvm::SmallVector<ModuleInfo, 16>::iterator I = Modules.begin(),<br>
-      E = Modules.end(); I != E; ++I) {<br>
-    ModuleInfo *MI = (ModuleInfo*)I;<br>
-    std::fprintf(stderr, "** %s\n", MI->FileName.c_str());<br>
-    if (MI->File)<br>
-      MI->File->dump();<br>
+  llvm::errs() << "*** Global Module Index Dump:\n";<br>
+  llvm::errs() << "Module files:\n";<br>
+  for (auto MI : Modules) {<br></blockquote><div><br></div><div>It doesn't make much difference in a dump() method, but this should be</div><div><br></div><div>  for (auto &MI : Modules) {</div><div><br></div><div>
... to avoid copying each ModuleInfo into the local variable.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    llvm::errs() << "** " << MI.FileName << "\n";<br>
+    if (MI.File)<br>
+      MI.File->dump();<br>
     else<br>
-      std::fprintf(stderr, "\n");<br>
+      llvm::errs() << "\n";<br>
   }<br>
-  std::fprintf(stderr, "\n");<br>
+  llvm::errs() << "\n";<br>
 }<br>
<br>
 //----------------------------------------------------------------------------//<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>