[PATCH] D147044: [clangd] Implement cross reference request for #include lines.

Mon Apr 3 06:39:35 PDT 2023

VitaNuo added a comment.

Thanks for the comments!

================
Comment at: clang-tools-extra/clangd/IncludeCleaner.h:87
+std::optional<include_cleaner::Header>
+firstMatchedProvider(const include_cleaner::Includes &Includes,
+                     llvm::ArrayRef<include_cleaner::Header> Providers);
----------------
kadircet wrote:
> can you also move tests related to this functionality from HoverTests into IncludeCleanerTests ?
Not really, not sure what you mean.

This function receives a ranked list of providers that should be a result of include_cleaner analysis.

If I just test the function based on a fake list of providers, the test will be trivial. The current hover test seems to provide more value than that.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147044