diff options
| author | David Blaikie <dblaikie@gmail.com> | 2013-05-14 21:04:00 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2013-05-14 21:04:00 +0000 |
| commit | d4da8728baeaf722ff0c53c8b8d3b49c53486111 (patch) | |
| tree | 077650393506345a7a2c767648824f47859c6a2f /clang/lib/AST/ASTDumper.cpp | |
| parent | 9fbc230de22d95bfc624571302c2738c994dd834 (diff) | |
| download | bcm5719-llvm-d4da8728baeaf722ff0c53c8b8d3b49c53486111.tar.gz bcm5719-llvm-d4da8728baeaf722ff0c53c8b8d3b49c53486111.zip | |
Provide operator<< for stream output of DeclarationNames
ASTDumper was already trying to do this & instead got an implicit bool
conversion by surprise (thus printing out 0 or 1 instead of the name of
the declaration). To avoid that issue & simplify call sites, simply make
it the normal/expected operator<<(raw_ostream&, ...) overload & simplify
all the existing call sites. (bonus: this function doesn't need to be a
member or friend, it's just using public API in DeclarationName)
llvm-svn: 181832
Diffstat (limited to 'clang/lib/AST/ASTDumper.cpp')
| -rw-r--r-- | clang/lib/AST/ASTDumper.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/clang/lib/AST/ASTDumper.cpp b/clang/lib/AST/ASTDumper.cpp index 340cc41f7e8..790538fcd06 100644 --- a/clang/lib/AST/ASTDumper.cpp +++ b/clang/lib/AST/ASTDumper.cpp @@ -449,9 +449,7 @@ void ASTDumper::dumpBareDeclRef(const Decl *D) { if (const NamedDecl *ND = dyn_cast<NamedDecl>(D)) { ColorScope Color(*this, DeclNameColor); - OS << " '"; - ND->getDeclName().printName(OS); - OS << "'"; + OS << " '" << ND->getDeclName() << '\''; } if (const ValueDecl *VD = dyn_cast<ValueDecl>(D)) |
