[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