[PATCH] D133499: [clang]: Add DeclContext::dumpAsDecl().

Tom Honermann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 1 04:52:32 PDT 2022


tahonermann marked an inline comment as done.
tahonermann added inline comments.


================
Comment at: clang/lib/AST/ASTDumper.cpp:203
 
+LLVM_DUMP_METHOD void DeclContext::dumpDecl() const {
+  if (const Decl *D = dyn_cast<Decl>(this))
----------------
shafik wrote:
> Interesting ` DeclContext::dumpDeclContext()` is in `DeclPrinter.cpp` I wonder why these were split like this.
It looks like the implementation of ` DeclContext::dumpDeclContext()` requires class `DeclPrinter` to be defined. `DeclPrinter` is defined in `clang/lib/AST/DeclPrinter.cpp` in an unnamed namespace.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133499/new/

https://reviews.llvm.org/D133499



More information about the cfe-commits mailing list