[PATCH] D43550: [clangd] Not collect include headers for dynamic index for now.
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 21 04:29:46 PST 2018
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
As discussed offline, properly supporting IWYU pragma in the dynamic index seems like too much design work for now and it's not gonna get fixed soon.
So opting for disabling the feature seems like the right approach.
Therefore, LGTM. (There are a few nits in inline comments from my previous reply that you might want to look at before landing this)
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D43550
More information about the cfe-commits
mailing list