maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Move VerifierSupport into namespace llvm. 

It currently is in an unnamed namespace and then it shouldn't be used
from something in the header file. This actually triggers a warning with
GCC:
../include/llvm/IR/Verifier.h:39:7: warning: ‘llvm::TBAAVerifier’ has a field ‘llvm::TBAAVerifier::Diagnostic’ whose type uses the anonymous namespace [enabled by default]

llvm-svn: 289942
This commit is contained in:
Daniel Jasper 2016-12-16 13:53:46 +00:00
parent 24bf8689dd
commit a44fd3014d
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
llvm/include/llvm/IR/Verifier.h
View File

@ -23,10 +23,6 @@
#include "llvm/IR/PassManager.h" #include "llvm/IR/PassManager.h"
namespace {
struct VerifierSupport;
}
namespace llvm { namespace llvm {
class Function; class Function;
@ -34,6 +30,7 @@ class FunctionPass;
class ModulePass; class ModulePass;
class Module; class Module;
class raw_ostream; class raw_ostream;
struct VerifierSupport;
/// Verify that the TBAA Metadatas are valid. /// Verify that the TBAA Metadatas are valid.
class TBAAVerifier { class TBAAVerifier {

6
llvm/lib/IR/Verifier.cpp
View File

@ -117,7 +117,7 @@ using namespace llvm;
static cl::opt<bool> VerifyDebugInfo("verify-debug-info", cl::init(true)); static cl::opt<bool> VerifyDebugInfo("verify-debug-info", cl::init(true));
namespace { namespace llvm {
struct VerifierSupport { struct VerifierSupport {
raw_ostream *OS; raw_ostream *OS;
@ -248,6 +248,10 @@ public:
} }
}; };
} // namespace llvm
namespace {
class Verifier : public InstVisitor<Verifier>, VerifierSupport { class Verifier : public InstVisitor<Verifier>, VerifierSupport {
friend class InstVisitor<Verifier>; friend class InstVisitor<Verifier>;