kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.
thanks, lgtm!
================
Comment at: clang-tools-extra/clangd/ParsedAST.cpp:270
+ // We don't want this, so keep track of them to fix afterwards.
+ llvm::DenseSet<diag::kind> NeedsWerrorExclusion;
+ for (diag::kind ID : Members) {
----------------
why do we need a set here, rather than a flag?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116147/new/
https://reviews.llvm.org/D116147
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
