[PATCH] D128204: [clangd] Add fix-it for inserting IWYU pragma: keep

[Nathan James via Phabricator via cfe-commits]

njames93 added a comment.

I had another idea about offering the export pragma when in a header file but I don't know if that's going too far

In D128204#3599286 <https://reviews.llvm.org/D128204#3599286>, @kadircet wrote:

> Hence i'd like to hear a little bit more about what kind of false positives you're facing often enough to need this as an automated fix at this stage, to see if there's something we can do to improve the implementation first.

One of the main issues I have is due to templates not being instantiated which can result in symbols that are actually used not being picked up as the template instantiation that uses them isn't actually instantiated.
Unfortunately this isn't something that can be easily fixed without tanking performance.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128204/new/

https://reviews.llvm.org/D128204