[PATCH] D143274: [clangd] Remove the direct use of StdSymbolMapping.inc usage.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 6 05:24:08 PST 2023


kadircet added inline comments.


================
Comment at: clang-tools-extra/clangd/index/CanonicalIncludes.h:43
+  /// Returns the overridden include for a qualified symbol with, or "".
+  /// \p Scope and \p Name concatenation forms the full qualified name.
+  /// \p Scope is the qualifier with the trailing "::" (e.g. "std::") or empty
----------------
s/full/fully


================
Comment at: clang-tools-extra/clangd/index/CanonicalIncludes.h:45
+  /// \p Scope is the qualifier with the trailing "::" (e.g. "std::") or empty
+  /// (for global namespace symbol).
+  llvm::StringRef mapSymbol(llvm::StringRef Scope, llvm::StringRef Name,
----------------
s/for global namespace symbol/for global namespace/


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143274



More information about the cfe-commits mailing list