[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