[lld] r264070 - Appease MSVC bots by changing visibility of AtomVector.

Pete Cooper via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 10:39:44 PDT 2016


Author: pete
Date: Tue Mar 22 12:39:44 2016
New Revision: 264070

URL: http://llvm.org/viewvc/llvm-project?rev=264070&view=rev
Log:
Appease MSVC bots by changing visibility of AtomVector.

The AtomVector class is an internal detail of File so I moved it
to be protected in r264067.  However, the MSVC bots don't like the
global declarations of type File::AtomVector in File.cpp so it needs
to go back to being public for now.

Modified:
    lld/trunk/include/lld/Core/File.h

Modified: lld/trunk/include/lld/Core/File.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/File.h?rev=264070&r1=264069&r2=264070&view=diff
==============================================================================
--- lld/trunk/include/lld/Core/File.h (original)
+++ lld/trunk/include/lld/Core/File.h Tue Mar 22 12:39:44 2016
@@ -40,10 +40,6 @@ class LinkingContext;
 /// The Atom objects in a File are owned by the File object.  The Atom objects
 /// are destroyed when the File object is destroyed.
 class File {
-protected:
-  /// The type of atom mutable container.
-  template <typename T> using AtomVector = std::vector<OwningAtomPtr<T>>;
-
 public:
   virtual ~File();
 
@@ -109,6 +105,9 @@ public:
     return _allocator;
   }
 
+  /// The type of atom mutable container.
+  template <typename T> using AtomVector = std::vector<OwningAtomPtr<T>>;
+
   /// The range type for the atoms.
   template <typename T> class AtomRange {
   public:




More information about the llvm-commits mailing list