[PATCH] D156650: [clangd] Respect IWYU keep pragma for standard headers.

hokein added a comment. In D156650#4545867 , @kadircet wrote: > also we should get this cherry-picked too. `keep` pragmas on includes are not > common, but people do have export pragmas often enough to cause some > annoyance here. Filed https://github.

[PATCH] D156650: [clangd] Respect IWYU keep pragma for standard headers.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGdcb28244faa8: [clangd] Respect IWYU keep pragma for standard headers. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D156650: [clangd] Respect IWYU keep pragma for standard headers.

hokein updated this revision to Diff 545589. hokein marked an inline comment as done. hokein added a comment. address review comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156650/new/ https://reviews.llvm.org/D156650 Files: clang-tools-ex

[PATCH] D156650: [clangd] Respect IWYU keep pragma for standard headers.

kadircet added a comment. also we should get this cherry-picked too. `keep` pragmas on includes are not common, but people do have export pragmas often enough to cause some annoyance here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156650/new/

[PATCH] D156650: [clangd] Respect IWYU keep pragma for standard headers.

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks! Comment at: clang-tools-extra/clangd/IncludeCleaner.cpp:85-86 if (PI) { if (PI->shouldKeep(Inc.HashLine + 1)) return false; // Check if main f

[PATCH] D156650: [clangd] Respect IWYU keep pragma for standard headers.

hokein created this revision. hokein added a reviewer: kadircet. Herald added a subscriber: arphaman. Herald added a project: All. hokein requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. see the issue https://github.com