replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[COFF] Avoid forward declaring StringSet, fix build 

This should fix the build after SVN r316178, which worked fine
on GCC 5.4, but failed on clang with errors like these:

MinGW.h:26:3: error: too few template arguments for class template 'StringSet'
  StringSet<> ExcludeSymbols;
  ^
lld/Common/LLVM.h:28:30: note: template is declared here
  template<typename T> class StringSet;

Don't forward declare and add the using directive in the main
lld/Common/LLVM.h header, but just qualify the class name
in MinGW.h instead.

llvm-svn: 316180
This commit is contained in:
Martin Storsjo 2017-10-19 20:19:16 +00:00
parent 769095bd07
commit 3af5ffb538
2 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lld/COFF/MinGW.h
View File

@ -23,9 +23,9 @@ class AutoExporter {
public:
AutoExporter();
StringSet<> ExcludeSymbols;
StringSet<> ExcludeLibs;
StringSet<> ExcludeObjects;
llvm::StringSet<> ExcludeSymbols;
llvm::StringSet<> ExcludeLibs;
llvm::StringSet<> ExcludeObjects;
bool shouldExport(Defined *Sym) const;
};

2
lld/include/lld/Common/LLVM.h
View File

@ -25,7 +25,6 @@ namespace llvm {
// ADT's.
class Error;
class StringRef;
template<typename T> class StringSet;
class Twine;
class MemoryBuffer;
class MemoryBufferRef;
@ -58,7 +57,6 @@ namespace lld {
// ADT's.
using llvm::Error;
using llvm::StringRef;
using llvm::StringSet;
using llvm::Twine;
using llvm::MemoryBuffer;
using llvm::MemoryBufferRef;