[PATCH] D148143: [clangd] Treat preamble patch as main file for include-cleaner analysis

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 12 11:27:16 PDT 2023


sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/clangd/Preamble.h:144
+  /// any.
+  static const FileEntry *getPatchEntry(llvm::StringRef MainFilePath,
+                                        const SourceManager &SM);
----------------
up to you, but this could be tested I think (just make sure it points to something valid with the right name when you have a patch and not otherwise)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148143



More information about the cfe-commits mailing list