[PATCH] D56641: NFC: Move dumping of QualType node to TextNodeDumper
This revision was automatically updated to reflect the committed changes. Closed by commit rL351116: [ASTDump] NFC: Move dumping of QualType node to TextNodeDumper (authored by steveire, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56641/new/ https://reviews.llvm.org/D56641 Files: cfe/trunk/include/clang/AST/TextNodeDumper.h cfe/trunk/lib/AST/ASTDumper.cpp cfe/trunk/lib/AST/TextNodeDumper.cpp Index: cfe/trunk/include/clang/AST/TextNodeDumper.h === --- cfe/trunk/include/clang/AST/TextNodeDumper.h +++ cfe/trunk/include/clang/AST/TextNodeDumper.h @@ -161,6 +161,8 @@ void Visit(const Type *T); + void Visit(QualType T); + void dumpPointer(const void *Ptr); void dumpLocation(SourceLocation Loc); void dumpSourceRange(SourceRange R); Index: cfe/trunk/lib/AST/TextNodeDumper.cpp === --- cfe/trunk/lib/AST/TextNodeDumper.cpp +++ cfe/trunk/lib/AST/TextNodeDumper.cpp @@ -172,6 +172,14 @@ OS << " imported"; } +void TextNodeDumper::Visit(QualType T) { + OS << "QualType"; + dumpPointer(T.getAsOpaquePtr()); + OS << " "; + dumpBareType(T, false); + OS << " " << T.split().Quals.getAsString(); +} + void TextNodeDumper::dumpPointer(const void *Ptr) { ColorScope Color(OS, ShowColors, AddressColor); OS << ' ' << Ptr; Index: cfe/trunk/lib/AST/ASTDumper.cpp === --- cfe/trunk/lib/AST/ASTDumper.cpp +++ cfe/trunk/lib/AST/ASTDumper.cpp @@ -422,11 +422,7 @@ return dumpTypeAsChild(SQT.Ty); dumpChild([=] { -OS << "QualType"; -NodeDumper.dumpPointer(T.getAsOpaquePtr()); -OS << " "; -NodeDumper.dumpBareType(T, false); -OS << " " << T.split().Quals.getAsString(); +NodeDumper.Visit(T); dumpTypeAsChild(T.split().Ty); }); } Index: cfe/trunk/include/clang/AST/TextNodeDumper.h === --- cfe/trunk/include/clang/AST/TextNodeDumper.h +++ cfe/trunk/include/clang/AST/TextNodeDumper.h @@ -161,6 +161,8 @@ void Visit(const Type *T); + void Visit(QualType T); + void dumpPointer(const void *Ptr); void dumpLocation(SourceLocation Loc); void dumpSourceRange(SourceRange R); Index: cfe/trunk/lib/AST/TextNodeDumper.cpp === --- cfe/trunk/lib/AST/TextNodeDumper.cpp +++ cfe/trunk/lib/AST/TextNodeDumper.cpp @@ -172,6 +172,14 @@ OS << " imported"; } +void TextNodeDumper::Visit(QualType T) { + OS << "QualType"; + dumpPointer(T.getAsOpaquePtr()); + OS << " "; + dumpBareType(T, false); + OS << " " << T.split().Quals.getAsString(); +} + void TextNodeDumper::dumpPointer(const void *Ptr) { ColorScope Color(OS, ShowColors, AddressColor); OS << ' ' << Ptr; Index: cfe/trunk/lib/AST/ASTDumper.cpp === --- cfe/trunk/lib/AST/ASTDumper.cpp +++ cfe/trunk/lib/AST/ASTDumper.cpp @@ -422,11 +422,7 @@ return dumpTypeAsChild(SQT.Ty); dumpChild([=] { -OS << "QualType"; -NodeDumper.dumpPointer(T.getAsOpaquePtr()); -OS << " "; -NodeDumper.dumpBareType(T, false); -OS << " " << T.split().Quals.getAsString(); +NodeDumper.Visit(T); dumpTypeAsChild(T.split().Ty); }); } ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D56641: NFC: Move dumping of QualType node to TextNodeDumper
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56641/new/ https://reviews.llvm.org/D56641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D56641: NFC: Move dumping of QualType node to TextNodeDumper
steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D56641 Files: include/clang/AST/TextNodeDumper.h lib/AST/ASTDumper.cpp lib/AST/TextNodeDumper.cpp Index: lib/AST/TextNodeDumper.cpp === --- lib/AST/TextNodeDumper.cpp +++ lib/AST/TextNodeDumper.cpp @@ -172,6 +172,14 @@ OS << " imported"; } +void TextNodeDumper::Visit(QualType T) { + OS << "QualType"; + dumpPointer(T.getAsOpaquePtr()); + OS << " "; + dumpBareType(T, false); + OS << " " << T.split().Quals.getAsString(); +} + void TextNodeDumper::dumpPointer(const void *Ptr) { ColorScope Color(OS, ShowColors, AddressColor); OS << ' ' << Ptr; Index: lib/AST/ASTDumper.cpp === --- lib/AST/ASTDumper.cpp +++ lib/AST/ASTDumper.cpp @@ -422,11 +422,7 @@ return dumpTypeAsChild(SQT.Ty); dumpChild([=] { -OS << "QualType"; -NodeDumper.dumpPointer(T.getAsOpaquePtr()); -OS << " "; -NodeDumper.dumpBareType(T, false); -OS << " " << T.split().Quals.getAsString(); +NodeDumper.Visit(T); dumpTypeAsChild(T.split().Ty); }); } Index: include/clang/AST/TextNodeDumper.h === --- include/clang/AST/TextNodeDumper.h +++ include/clang/AST/TextNodeDumper.h @@ -161,6 +161,8 @@ void Visit(const Type *T); + void Visit(QualType T); + void dumpPointer(const void *Ptr); void dumpLocation(SourceLocation Loc); void dumpSourceRange(SourceRange R); Index: lib/AST/TextNodeDumper.cpp === --- lib/AST/TextNodeDumper.cpp +++ lib/AST/TextNodeDumper.cpp @@ -172,6 +172,14 @@ OS << " imported"; } +void TextNodeDumper::Visit(QualType T) { + OS << "QualType"; + dumpPointer(T.getAsOpaquePtr()); + OS << " "; + dumpBareType(T, false); + OS << " " << T.split().Quals.getAsString(); +} + void TextNodeDumper::dumpPointer(const void *Ptr) { ColorScope Color(OS, ShowColors, AddressColor); OS << ' ' << Ptr; Index: lib/AST/ASTDumper.cpp === --- lib/AST/ASTDumper.cpp +++ lib/AST/ASTDumper.cpp @@ -422,11 +422,7 @@ return dumpTypeAsChild(SQT.Ty); dumpChild([=] { -OS << "QualType"; -NodeDumper.dumpPointer(T.getAsOpaquePtr()); -OS << " "; -NodeDumper.dumpBareType(T, false); -OS << " " << T.split().Quals.getAsString(); +NodeDumper.Visit(T); dumpTypeAsChild(T.split().Ty); }); } Index: include/clang/AST/TextNodeDumper.h === --- include/clang/AST/TextNodeDumper.h +++ include/clang/AST/TextNodeDumper.h @@ -161,6 +161,8 @@ void Visit(const Type *T); + void Visit(QualType T); + void dumpPointer(const void *Ptr); void dumpLocation(SourceLocation Loc); void dumpSourceRange(SourceRange R); ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits