[PATCH] D112571: [clangd] IncludeCleaner: Don't warn on system headers

2021-10-27 Thread Kirill Bobyrev via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc4723785c190: [clangd] IncludeCleaner: Don't warn on system headers (authored by kbobyrev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D112571: [clangd] IncludeCleaner: Don't warn on system headers

2021-10-27 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev updated this revision to Diff 382576. kbobyrev marked an inline comment as done. kbobyrev added a comment. Resolve review comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112571/new/ https://reviews.llvm.org/D112571 Files: clang-t

[PATCH] D112571: [clangd] IncludeCleaner: Don't warn on system headers

2021-10-27 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/IncludeCleaner.cpp:258 for (const auto *Inc : computeUnusedIncludes(AST)) { +// FIXME(kirillbobyrev): Standard library h

[PATCH] D112571: [clangd] IncludeCleaner: Don't warn on system headers

2021-10-26 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev created this revision. kbobyrev added a reviewer: sammccall. Herald added subscribers: usaxena95, arphaman. kbobyrev requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. This is a temporary hack to dis