[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