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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
