iains abandoned this revision.
iains added a comment.
although this was approved, we did not need to use it to implement the
dependent changes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126691/new/
https://reviews.llvm.org/D126691
___
ChuanqiXu accepted this revision.
ChuanqiXu added a comment.
This revision is now accepted and ready to land.
LGTM with a nits.
Comment at: clang/include/clang/AST/ASTContext.h:1083
+ /// Does the module initializer array contain this decl.
+ bool moduleInitializerContains(Mo
iains created this revision.
Herald added a project: All.
iains added reviewers: urnathan, ChuanqiXu.
iains published this revision for review.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
When deciding if a global module fragement decl is unused, we have
to account for t
