[PATCH] D80878: [clang] Prevent that Decl::dump on a CXXRecordDecl deserialises further declarations.

Volodymyr Sapsai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 4 12:57:06 PDT 2021


vsapsai added inline comments.


================
Comment at: clang/lib/AST/ASTDumper.cpp:199
     P.setDeserialize(Deserialize);
+    P.doGetNodeDelegate().setDeserialize(Deserialize);
     P.Visit(this);
----------------
It is possible it will complicate the implementation too much but what if `P.setDeserialize(Deserialize);` is responsible for propagating `Deserialize` to its NodeDelegate?


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

https://reviews.llvm.org/D80878



More information about the cfe-commits mailing list