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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to