[PATCH] D158055: [clang][AST] Added some missing setter methods
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 16 11:41:06 PDT 2023
aaron.ballman added a comment.
We typically only add these sorts of functions when we need them; do you have a need to add setters? (Oftentimes, we don't want setters because we want the AST nodes to be created in the correct state when possible; we do add some setters because of AST deserialization where we need to create an empty shell AST node and fill it out piecewise as we deserialize.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158055/new/
https://reviews.llvm.org/D158055
More information about the cfe-commits
mailing list