commit 467209b1d4763f77a7945f5572375a17ed179287
author Nick Kledzik <kledzik@apple.com> Mon Apr 09 00:58:21 2012 +0000
committer Nick Kledzik <kledzik@apple.com> Mon Apr 09 00:58:21 2012 +0000
tree b78a3fa3fcbcee3635c1a7a55bef7a0f805ca175
parent 062a98cff019bec87be5ee9f4993bd29f46c9942

Remove definedAtomsBegin() and co. so that C++11 range based for loops can be used

llvm-svn: 154302

lld/lib/Core/YamlWriter.cpp

diff --git a/lld/lib/Core/YamlWriter.cpp b/lld/lib/Core/YamlWriter.cpp
index db63b61..c5f45c9 100644
--- a/lld/lib/Core/YamlWriter.cpp
+++ b/lld/lib/Core/YamlWriter.cpp
@@ -48,10 +48,7 @@
   RefNameBuilder(const File& file)
                 : _collisionCount(0), _unnamedCounter(0) {
     // visit all atoms
-    for(File::defined_iterator it=file.definedAtomsBegin(),
-                              end=file.definedAtomsEnd();
-                               it != end; ++it) {
-      const DefinedAtom* atom = *it;
+    for( const DefinedAtom *atom : file.defined() ) {
       // Build map of atoms names to detect duplicates
       if ( ! atom->name().empty() )
         buildDuplicateNameMap(*atom);
@@ -67,20 +64,14 @@
         }
       }
     }
-    for(File::undefined_iterator it=file.undefinedAtomsBegin(),
-                              end=file.undefinedAtomsEnd();
-                               it != end; ++it) {
-      buildDuplicateNameMap(**it);
+    for( const UndefinedAtom *undefAtom : file.undefined() ) {
+      buildDuplicateNameMap(*undefAtom);
     }
-    for(File::shared_library_iterator it=file.sharedLibraryAtomsBegin(),
-                              end=file.sharedLibraryAtomsEnd();
-                               it != end; ++it) {
-      buildDuplicateNameMap(**it);
+    for( const SharedLibraryAtom *shlibAtom : file.sharedLibrary() ) {
+      buildDuplicateNameMap(*shlibAtom);
     }
-    for(File::absolute_iterator it=file.absoluteAtomsBegin(),
-                              end=file.absoluteAtomsEnd();
-                               it != end; ++it) {
-      buildDuplicateNameMap(**it);
+    for( const AbsoluteAtom *absAtom : file.absolute() ) {
+      buildDuplicateNameMap(*absAtom);
     }
 
 
@@ -142,25 +133,17 @@
     out << "---\n";
 
     // visit all atoms
-    for(File::defined_iterator it=_file.definedAtomsBegin(),
-                              end=_file.definedAtomsEnd();
-                               it != end; ++it) {
-      writeDefinedAtom(**it, out);
+    for( const DefinedAtom *atom : _file.defined() ) {
+      writeDefinedAtom(*atom, out);
     }
-    for(File::undefined_iterator it=_file.undefinedAtomsBegin(),
-                              end=_file.undefinedAtomsEnd();
-                               it != end; ++it) {
-      writeUndefinedAtom(**it, out);
+    for( const UndefinedAtom *undefAtom : _file.undefined() ) {
+      writeUndefinedAtom(*undefAtom, out);
     }
-    for(File::shared_library_iterator it=_file.sharedLibraryAtomsBegin(),
-                              end=_file.sharedLibraryAtomsEnd();
-                               it != end; ++it) {
-      writeSharedLibraryAtom(**it, out);
+    for( const SharedLibraryAtom *shlibAtom : _file.sharedLibrary() ) {
+      writeSharedLibraryAtom(*shlibAtom, out);
     }
-    for(File::absolute_iterator it=_file.absoluteAtomsBegin(),
-                              end=_file.absoluteAtomsEnd();
-                               it != end; ++it) {
-      writeAbsoluteAtom(**it, out);
+    for( const AbsoluteAtom *absAtom : _file.absolute() ) {
+      writeAbsoluteAtom(*absAtom, out);
     }
 
     out << "...\n";