[PATCH] D75560: Make Decl:: setOwningModuleID() public. (NFC)

2020-03-11 Thread Adrian Prantl via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG2b366e75d413: Make Decl::setOwningModuleID() public. (NFC) (authored by aprantl). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D75560: Make Decl:: setOwningModuleID() public. (NFC)

2020-03-11 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik accepted this revision. shafik added a comment. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75560/new/ https://reviews.llvm.org/D75560 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mail

[PATCH] D75560: Make Decl:: setOwningModuleID() public. (NFC)

2020-03-05 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. I'm surprised so few setters needed to be made public for this to work. If this required making lots more setters public, I'd be concerned, because we generally don't want large chunks of the

[PATCH] D75560: Make Decl:: setOwningModuleID() public. (NFC)

2020-03-05 Thread Adrian Prantl via Phabricator via cfe-commits
aprantl marked an inline comment as done. aprantl added inline comments. Comment at: clang/include/clang/AST/DeclBase.h:629-630 +public: + void setFromASTFile() { FromASTFile = true; } + rsmith wrote: > Setting this after creating a `Decl` is not safe in gener

[PATCH] D75560: Make Decl:: setOwningModuleID() public. (NFC)

2020-03-05 Thread Adrian Prantl via Phabricator via cfe-commits
aprantl updated this revision to Diff 248618. aprantl added a comment. I took your suggestion to heart and changed the downstream LLDB patch to use CreateDeserialized() everywhere, which is frankly a better fit there anyway. To make this possible I had to make a few more setters public, see the

[PATCH] D75560: Make Decl:: setOwningModuleID() public. (NFC)

2020-03-04 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added inline comments. Comment at: clang/include/clang/AST/DeclBase.h:629-630 +public: + void setFromASTFile() { FromASTFile = true; } + Setting this after creating a `Decl` is not safe in general; declarations with this flag set have 8 bytes of additi