[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