blob: c7428d850489d21bbdc26cb225f11f24b387748b [file] [log] [blame]
Haojian Wu4c1394d2017-12-12 15:42:10 +00001//===--- SymbolCollector.cpp -------------------------------------*- C++-*-===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9
10#include "SymbolCollector.h"
Haojian Wu5f100262018-03-09 14:00:34 +000011#include "../AST.h"
Eric Liu76f6b442018-01-09 17:32:00 +000012#include "../CodeCompletionStrings.h"
Eric Liu7f247652018-02-06 16:10:35 +000013#include "../Logger.h"
Marc-Andre Laperleb387b6e2018-04-23 20:00:52 +000014#include "../SourceCode.h"
Eric Liu7f247652018-02-06 16:10:35 +000015#include "../URI.h"
Eric Liuc5105f92018-02-16 14:15:55 +000016#include "CanonicalIncludes.h"
Haojian Wu4c1394d2017-12-12 15:42:10 +000017#include "clang/AST/DeclCXX.h"
Ilya Biryukovcf124bd2018-04-13 11:03:07 +000018#include "clang/AST/DeclTemplate.h"
Eric Liu9af958f2018-01-10 14:57:58 +000019#include "clang/ASTMatchers/ASTMatchFinder.h"
Haojian Wu4c1394d2017-12-12 15:42:10 +000020#include "clang/Basic/SourceManager.h"
21#include "clang/Index/IndexSymbol.h"
22#include "clang/Index/USRGeneration.h"
Eric Liu278e2d12018-01-29 15:13:29 +000023#include "llvm/Support/FileSystem.h"
Haojian Wu4c1394d2017-12-12 15:42:10 +000024#include "llvm/Support/MemoryBuffer.h"
25#include "llvm/Support/Path.h"
26
27namespace clang {
28namespace clangd {
29
30namespace {
Ilya Biryukovf118d512018-04-14 16:27:35 +000031/// If \p ND is a template specialization, returns the described template.
Ilya Biryukovcf124bd2018-04-13 11:03:07 +000032/// Otherwise, returns \p ND.
33const NamedDecl &getTemplateOrThis(const NamedDecl &ND) {
Ilya Biryukovf118d512018-04-14 16:27:35 +000034 if (auto T = ND.getDescribedTemplate())
35 return *T;
Ilya Biryukovcf124bd2018-04-13 11:03:07 +000036 return ND;
37}
38
Eric Liu7f247652018-02-06 16:10:35 +000039// Returns a URI of \p Path. Firstly, this makes the \p Path absolute using the
40// current working directory of the given SourceManager if the Path is not an
41// absolute path. If failed, this resolves relative paths against \p FallbackDir
42// to get an absolute path. Then, this tries creating an URI for the absolute
43// path with schemes specified in \p Opts. This returns an URI with the first
44// working scheme, if there is any; otherwise, this returns None.
Haojian Wu4c1394d2017-12-12 15:42:10 +000045//
46// The Path can be a path relative to the build directory, or retrieved from
47// the SourceManager.
Eric Liu7f247652018-02-06 16:10:35 +000048llvm::Optional<std::string> toURI(const SourceManager &SM, StringRef Path,
49 const SymbolCollector::Options &Opts) {
Haojian Wu4c1394d2017-12-12 15:42:10 +000050 llvm::SmallString<128> AbsolutePath(Path);
51 if (std::error_code EC =
52 SM.getFileManager().getVirtualFileSystem()->makeAbsolute(
53 AbsolutePath))
Eric Liu20defe12018-05-15 16:22:43 +000054 log("Warning: could not make absolute file: " + EC.message());
Eric Liu278e2d12018-01-29 15:13:29 +000055 if (llvm::sys::path::is_absolute(AbsolutePath)) {
56 // Handle the symbolic link path case where the current working directory
57 // (getCurrentWorkingDirectory) is a symlink./ We always want to the real
58 // file path (instead of the symlink path) for the C++ symbols.
59 //
60 // Consider the following example:
61 //
62 // src dir: /project/src/foo.h
63 // current working directory (symlink): /tmp/build -> /project/src/
64 //
65 // The file path of Symbol is "/project/src/foo.h" instead of
66 // "/tmp/build/foo.h"
67 if (const DirectoryEntry *Dir = SM.getFileManager().getDirectory(
68 llvm::sys::path::parent_path(AbsolutePath.str()))) {
69 StringRef DirName = SM.getFileManager().getCanonicalName(Dir);
70 SmallString<128> AbsoluteFilename;
71 llvm::sys::path::append(AbsoluteFilename, DirName,
72 llvm::sys::path::filename(AbsolutePath.str()));
73 AbsolutePath = AbsoluteFilename;
74 }
Eric Liu7f247652018-02-06 16:10:35 +000075 } else if (!Opts.FallbackDir.empty()) {
76 llvm::sys::fs::make_absolute(Opts.FallbackDir, AbsolutePath);
Eric Liu278e2d12018-01-29 15:13:29 +000077 llvm::sys::path::remove_dots(AbsolutePath, /*remove_dot_dot=*/true);
Haojian Wu4c1394d2017-12-12 15:42:10 +000078 }
Eric Liu7f247652018-02-06 16:10:35 +000079
80 std::string ErrMsg;
81 for (const auto &Scheme : Opts.URISchemes) {
82 auto U = URI::create(AbsolutePath, Scheme);
83 if (U)
84 return U->toString();
85 ErrMsg += llvm::toString(U.takeError()) + "\n";
86 }
87 log(llvm::Twine("Failed to create an URI for file ") + AbsolutePath + ": " +
88 ErrMsg);
89 return llvm::None;
Haojian Wu4c1394d2017-12-12 15:42:10 +000090}
Eric Liu4feda802017-12-19 11:37:40 +000091
Eric Liud67ec242018-05-16 12:12:30 +000092// All proto generated headers should start with this line.
93static const char *PROTO_HEADER_COMMENT =
94 "// Generated by the protocol buffer compiler. DO NOT EDIT!";
95
96// Checks whether the decl is a private symbol in a header generated by
97// protobuf compiler.
98// To identify whether a proto header is actually generated by proto compiler,
99// we check whether it starts with PROTO_HEADER_COMMENT.
100// FIXME: make filtering extensible when there are more use cases for symbol
101// filters.
102bool isPrivateProtoDecl(const NamedDecl &ND) {
103 const auto &SM = ND.getASTContext().getSourceManager();
104 auto Loc = findNameLoc(&ND);
105 auto FileName = SM.getFilename(Loc);
106 if (!FileName.endswith(".proto.h") && !FileName.endswith(".pb.h"))
107 return false;
108 auto FID = SM.getFileID(Loc);
109 // Double check that this is an actual protobuf header.
110 if (!SM.getBufferData(FID).startswith(PROTO_HEADER_COMMENT))
111 return false;
112
113 // ND without identifier can be operators.
114 if (ND.getIdentifier() == nullptr)
115 return false;
116 auto Name = ND.getIdentifier()->getName();
117 if (!Name.contains('_'))
118 return false;
119 // Nested proto entities (e.g. Message::Nested) have top-level decls
120 // that shouldn't be used (Message_Nested). Ignore them completely.
121 // The nested entities are dangling type aliases, we may want to reconsider
122 // including them in the future.
123 // For enum constants, SOME_ENUM_CONSTANT is not private and should be
124 // indexed. Outer_INNER is private. This heuristic relies on naming style, it
125 // will include OUTER_INNER and exclude some_enum_constant.
126 // FIXME: the heuristic relies on naming style (i.e. no underscore in
127 // user-defined names) and can be improved.
128 return (ND.getKind() != Decl::EnumConstant) ||
129 std::any_of(Name.begin(), Name.end(), islower);
130}
131
Eric Liu9af958f2018-01-10 14:57:58 +0000132bool shouldFilterDecl(const NamedDecl *ND, ASTContext *ASTCtx,
133 const SymbolCollector::Options &Opts) {
134 using namespace clang::ast_matchers;
135 if (ND->isImplicit())
136 return true;
Haojian Wu9873fdd2018-01-19 09:35:55 +0000137 // Skip anonymous declarations, e.g (anonymous enum/class/struct).
138 if (ND->getDeclName().isEmpty())
139 return true;
140
Eric Liu9af958f2018-01-10 14:57:58 +0000141 // FIXME: figure out a way to handle internal linkage symbols (e.g. static
142 // variables, function) defined in the .cc files. Also we skip the symbols
143 // in anonymous namespace as the qualifier names of these symbols are like
144 // `foo::<anonymous>::bar`, which need a special handling.
145 // In real world projects, we have a relatively large set of header files
146 // that define static variables (like "static const int A = 1;"), we still
147 // want to collect these symbols, although they cause potential ODR
148 // violations.
149 if (ND->isInAnonymousNamespace())
150 return true;
151
Haojian Wu9873fdd2018-01-19 09:35:55 +0000152 // We only want:
153 // * symbols in namespaces or translation unit scopes (e.g. no class
154 // members)
155 // * enum constants in unscoped enum decl (e.g. "red" in "enum {red};")
Eric Liucf177382018-02-02 10:31:42 +0000156 auto InTopLevelScope = hasDeclContext(
157 anyOf(namespaceDecl(), translationUnitDecl(), linkageSpecDecl()));
Sam McCall824913b2018-03-09 13:25:29 +0000158 // Don't index template specializations.
159 auto IsSpecialization =
160 anyOf(functionDecl(isExplicitTemplateSpecialization()),
161 cxxRecordDecl(isExplicitTemplateSpecialization()),
162 varDecl(isExplicitTemplateSpecialization()));
Eric Liucf8601b2018-02-28 09:33:15 +0000163 if (match(decl(allOf(unless(isExpansionInMainFile()),
Haojian Wu9873fdd2018-01-19 09:35:55 +0000164 anyOf(InTopLevelScope,
165 hasDeclContext(enumDecl(InTopLevelScope,
Sam McCall824913b2018-03-09 13:25:29 +0000166 unless(isScoped())))),
167 unless(IsSpecialization))),
Eric Liu9af958f2018-01-10 14:57:58 +0000168 *ND, *ASTCtx)
169 .empty())
170 return true;
171
Eric Liud67ec242018-05-16 12:12:30 +0000172 // Avoid indexing internal symbols in protobuf generated headers.
173 if (isPrivateProtoDecl(*ND))
174 return true;
Eric Liu9af958f2018-01-10 14:57:58 +0000175 return false;
176}
177
Eric Liuc5105f92018-02-16 14:15:55 +0000178// We only collect #include paths for symbols that are suitable for global code
179// completion, except for namespaces since #include path for a namespace is hard
180// to define.
181bool shouldCollectIncludePath(index::SymbolKind Kind) {
182 using SK = index::SymbolKind;
183 switch (Kind) {
184 case SK::Macro:
185 case SK::Enum:
186 case SK::Struct:
187 case SK::Class:
188 case SK::Union:
189 case SK::TypeAlias:
190 case SK::Using:
191 case SK::Function:
192 case SK::Variable:
193 case SK::EnumConstant:
194 return true;
195 default:
196 return false;
197 }
198}
199
Eric Liu02ce01f2018-02-22 10:14:05 +0000200/// Gets a canonical include (URI of the header or <header> or "header") for
201/// header of \p Loc.
202/// Returns None if fails to get include header for \p Loc.
Eric Liuc5105f92018-02-16 14:15:55 +0000203/// FIXME: we should handle .inc files whose symbols are expected be exported by
204/// their containing headers.
205llvm::Optional<std::string>
Eric Liub96363d2018-03-01 18:06:40 +0000206getIncludeHeader(llvm::StringRef QName, const SourceManager &SM,
207 SourceLocation Loc, const SymbolCollector::Options &Opts) {
Eric Liuc5105f92018-02-16 14:15:55 +0000208 llvm::StringRef FilePath = SM.getFilename(Loc);
209 if (FilePath.empty())
210 return llvm::None;
211 if (Opts.Includes) {
Eric Liub96363d2018-03-01 18:06:40 +0000212 llvm::StringRef Mapped = Opts.Includes->mapHeader(FilePath, QName);
Eric Liuc5105f92018-02-16 14:15:55 +0000213 if (Mapped != FilePath)
214 return (Mapped.startswith("<") || Mapped.startswith("\""))
215 ? Mapped.str()
216 : ("\"" + Mapped + "\"").str();
217 }
Eric Liu02ce01f2018-02-22 10:14:05 +0000218
219 return toURI(SM, SM.getFilename(Loc), Opts);
Eric Liuc5105f92018-02-16 14:15:55 +0000220}
221
Haojian Wub0189062018-01-31 12:56:51 +0000222// Return the symbol location of the given declaration `D`.
223//
224// For symbols defined inside macros:
225// * use expansion location, if the symbol is formed via macro concatenation.
226// * use spelling location, otherwise.
Eric Liucf8601b2018-02-28 09:33:15 +0000227llvm::Optional<SymbolLocation> getSymbolLocation(
228 const NamedDecl &D, SourceManager &SM, const SymbolCollector::Options &Opts,
229 const clang::LangOptions &LangOpts, std::string &FileURIStorage) {
Haojian Wu5f100262018-03-09 14:00:34 +0000230 SourceLocation NameLoc = findNameLoc(&D);
231 auto U = toURI(SM, SM.getFilename(NameLoc), Opts);
Eric Liu7f247652018-02-06 16:10:35 +0000232 if (!U)
233 return llvm::None;
234 FileURIStorage = std::move(*U);
Sam McCall60039512018-02-09 14:42:01 +0000235 SymbolLocation Result;
236 Result.FileURI = FileURIStorage;
Haojian Wu545c02a2018-04-13 08:30:39 +0000237 auto TokenLength = clang::Lexer::MeasureTokenLength(NameLoc, SM, LangOpts);
238
239 auto CreatePosition = [&SM](SourceLocation Loc) {
Haojian Wuc5340012018-04-30 11:40:02 +0000240 auto LSPLoc = sourceLocToPosition(SM, Loc);
Haojian Wu545c02a2018-04-13 08:30:39 +0000241 SymbolLocation::Position Pos;
Haojian Wuc5340012018-04-30 11:40:02 +0000242 Pos.Line = LSPLoc.line;
243 Pos.Column = LSPLoc.character;
Haojian Wu545c02a2018-04-13 08:30:39 +0000244 return Pos;
245 };
246
247 Result.Start = CreatePosition(NameLoc);
248 auto EndLoc = NameLoc.getLocWithOffset(TokenLength);
249 Result.End = CreatePosition(EndLoc);
250
Sam McCall60039512018-02-09 14:42:01 +0000251 return std::move(Result);
Haojian Wub0189062018-01-31 12:56:51 +0000252}
253
Eric Liucf8601b2018-02-28 09:33:15 +0000254// Checks whether \p ND is a definition of a TagDecl (class/struct/enum/union)
255// in a header file, in which case clangd would prefer to use ND as a canonical
256// declaration.
257// FIXME: handle symbol types that are not TagDecl (e.g. functions), if using
Fangrui Song943e12e2018-03-29 20:03:16 +0000258// the first seen declaration as canonical declaration is not a good enough
Eric Liucf8601b2018-02-28 09:33:15 +0000259// heuristic.
260bool isPreferredDeclaration(const NamedDecl &ND, index::SymbolRoleSet Roles) {
261 using namespace clang::ast_matchers;
262 return (Roles & static_cast<unsigned>(index::SymbolRole::Definition)) &&
263 llvm::isa<TagDecl>(&ND) &&
264 match(decl(isExpansionInMainFile()), ND, ND.getASTContext()).empty();
265}
266
Haojian Wu4c1394d2017-12-12 15:42:10 +0000267} // namespace
268
Eric Liu9af958f2018-01-10 14:57:58 +0000269SymbolCollector::SymbolCollector(Options Opts) : Opts(std::move(Opts)) {}
270
Eric Liu76f6b442018-01-09 17:32:00 +0000271void SymbolCollector::initialize(ASTContext &Ctx) {
272 ASTCtx = &Ctx;
273 CompletionAllocator = std::make_shared<GlobalCodeCompletionAllocator>();
274 CompletionTUInfo =
275 llvm::make_unique<CodeCompletionTUInfo>(CompletionAllocator);
276}
277
Haojian Wu4c1394d2017-12-12 15:42:10 +0000278// Always return true to continue indexing.
279bool SymbolCollector::handleDeclOccurence(
280 const Decl *D, index::SymbolRoleSet Roles,
Sam McCallb9d57112018-04-09 14:28:52 +0000281 ArrayRef<index::SymbolRelation> Relations, SourceLocation Loc,
Haojian Wu4c1394d2017-12-12 15:42:10 +0000282 index::IndexDataConsumer::ASTNodeInfo ASTNode) {
Eric Liu9af958f2018-01-10 14:57:58 +0000283 assert(ASTCtx && PP.get() && "ASTContext and Preprocessor must be set.");
Sam McCall93f99bf2018-03-12 14:49:09 +0000284 assert(CompletionAllocator && CompletionTUInfo);
285 const NamedDecl *ND = llvm::dyn_cast<NamedDecl>(D);
286 if (!ND)
287 return true;
Eric Liu9af958f2018-01-10 14:57:58 +0000288
Sam McCall93f99bf2018-03-12 14:49:09 +0000289 // Mark D as referenced if this is a reference coming from the main file.
290 // D may not be an interesting symbol, but it's cheaper to check at the end.
Sam McCallb9d57112018-04-09 14:28:52 +0000291 auto &SM = ASTCtx->getSourceManager();
Sam McCall93f99bf2018-03-12 14:49:09 +0000292 if (Opts.CountReferences &&
293 (Roles & static_cast<unsigned>(index::SymbolRole::Reference)) &&
Sam McCallb9d57112018-04-09 14:28:52 +0000294 SM.getFileID(SM.getSpellingLoc(Loc)) == SM.getMainFileID())
Sam McCall93f99bf2018-03-12 14:49:09 +0000295 ReferencedDecls.insert(ND);
296
297 // Don't continue indexing if this is a mere reference.
Haojian Wu4c1394d2017-12-12 15:42:10 +0000298 if (!(Roles & static_cast<unsigned>(index::SymbolRole::Declaration) ||
299 Roles & static_cast<unsigned>(index::SymbolRole::Definition)))
300 return true;
Sam McCall93f99bf2018-03-12 14:49:09 +0000301 if (shouldFilterDecl(ND, ASTCtx, Opts))
302 return true;
Haojian Wu4c1394d2017-12-12 15:42:10 +0000303
Sam McCall93f99bf2018-03-12 14:49:09 +0000304 llvm::SmallString<128> USR;
305 if (index::generateUSRForDecl(ND, USR))
306 return true;
307 SymbolID ID(USR);
Eric Liu76f6b442018-01-09 17:32:00 +0000308
Sam McCall93f99bf2018-03-12 14:49:09 +0000309 const NamedDecl &OriginalDecl = *cast<NamedDecl>(ASTNode.OrigD);
310 const Symbol *BasicSymbol = Symbols.find(ID);
311 if (!BasicSymbol) // Regardless of role, ND is the canonical declaration.
312 BasicSymbol = addDeclaration(*ND, std::move(ID));
313 else if (isPreferredDeclaration(OriginalDecl, Roles))
314 // If OriginalDecl is preferred, replace the existing canonical
315 // declaration (e.g. a class forward declaration). There should be at most
316 // one duplicate as we expect to see only one preferred declaration per
317 // TU, because in practice they are definitions.
318 BasicSymbol = addDeclaration(OriginalDecl, std::move(ID));
Haojian Wu4c1394d2017-12-12 15:42:10 +0000319
Sam McCall93f99bf2018-03-12 14:49:09 +0000320 if (Roles & static_cast<unsigned>(index::SymbolRole::Definition))
321 addDefinition(OriginalDecl, *BasicSymbol);
Haojian Wu4c1394d2017-12-12 15:42:10 +0000322 return true;
323}
324
Sam McCall93f99bf2018-03-12 14:49:09 +0000325void SymbolCollector::finish() {
326 // At the end of the TU, add 1 to the refcount of the ReferencedDecls.
327 for (const auto *ND : ReferencedDecls) {
328 llvm::SmallString<128> USR;
329 if (!index::generateUSRForDecl(ND, USR))
330 if (const auto *S = Symbols.find(SymbolID(USR))) {
331 Symbol Inc = *S;
332 ++Inc.References;
333 Symbols.insert(Inc);
334 }
335 }
336 ReferencedDecls.clear();
337}
338
Sam McCall60039512018-02-09 14:42:01 +0000339const Symbol *SymbolCollector::addDeclaration(const NamedDecl &ND,
340 SymbolID ID) {
Ilya Biryukov43714502018-05-16 12:32:44 +0000341 auto &Ctx = ND.getASTContext();
342 auto &SM = Ctx.getSourceManager();
Sam McCall60039512018-02-09 14:42:01 +0000343
344 std::string QName;
345 llvm::raw_string_ostream OS(QName);
346 PrintingPolicy Policy(ASTCtx->getLangOpts());
347 // Note that inline namespaces are treated as transparent scopes. This
348 // reflects the way they're most commonly used for lookup. Ideally we'd
349 // include them, but at query time it's hard to find all the inline
350 // namespaces to query: the preamble doesn't have a dedicated list.
351 Policy.SuppressUnwrittenScope = true;
352 ND.printQualifiedName(OS, Policy);
353 OS.flush();
Marc-Andre Laperleb387b6e2018-04-23 20:00:52 +0000354 assert(!StringRef(QName).startswith("::"));
Sam McCall60039512018-02-09 14:42:01 +0000355
356 Symbol S;
357 S.ID = std::move(ID);
358 std::tie(S.Scope, S.Name) = splitQualifiedName(QName);
359 S.SymInfo = index::getSymbolInfo(&ND);
360 std::string FileURI;
Haojian Wudc02a3d2018-02-13 09:53:50 +0000361 if (auto DeclLoc =
362 getSymbolLocation(ND, SM, Opts, ASTCtx->getLangOpts(), FileURI))
Sam McCall60039512018-02-09 14:42:01 +0000363 S.CanonicalDeclaration = *DeclLoc;
364
365 // Add completion info.
366 // FIXME: we may want to choose a different redecl, or combine from several.
367 assert(ASTCtx && PP.get() && "ASTContext and Preprocessor must be set.");
Ilya Biryukovcf124bd2018-04-13 11:03:07 +0000368 // We use the primary template, as clang does during code completion.
369 CodeCompletionResult SymbolCompletion(&getTemplateOrThis(ND), 0);
Sam McCall60039512018-02-09 14:42:01 +0000370 const auto *CCS = SymbolCompletion.CreateCodeCompletionString(
371 *ASTCtx, *PP, CodeCompletionContext::CCC_Name, *CompletionAllocator,
372 *CompletionTUInfo,
Ilya Biryukov43714502018-05-16 12:32:44 +0000373 /*IncludeBriefComments*/ false);
Sam McCall60039512018-02-09 14:42:01 +0000374 std::string Label;
375 std::string SnippetInsertText;
376 std::string IgnoredLabel;
377 std::string PlainInsertText;
378 getLabelAndInsertText(*CCS, &Label, &SnippetInsertText,
379 /*EnableSnippets=*/true);
380 getLabelAndInsertText(*CCS, &IgnoredLabel, &PlainInsertText,
381 /*EnableSnippets=*/false);
382 std::string FilterText = getFilterText(*CCS);
Ilya Biryukov43714502018-05-16 12:32:44 +0000383 std::string Documentation =
384 formatDocumentation(*CCS, getDocComment(Ctx, SymbolCompletion));
Sam McCall60039512018-02-09 14:42:01 +0000385 std::string CompletionDetail = getDetail(*CCS);
386
Eric Liuc5105f92018-02-16 14:15:55 +0000387 std::string Include;
388 if (Opts.CollectIncludePath && shouldCollectIncludePath(S.SymInfo.Kind)) {
389 // Use the expansion location to get the #include header since this is
390 // where the symbol is exposed.
Eric Liub96363d2018-03-01 18:06:40 +0000391 if (auto Header = getIncludeHeader(
392 QName, SM, SM.getExpansionLoc(ND.getLocation()), Opts))
Eric Liuc5105f92018-02-16 14:15:55 +0000393 Include = std::move(*Header);
394 }
Sam McCall60039512018-02-09 14:42:01 +0000395 S.CompletionFilterText = FilterText;
396 S.CompletionLabel = Label;
397 S.CompletionPlainInsertText = PlainInsertText;
398 S.CompletionSnippetInsertText = SnippetInsertText;
399 Symbol::Details Detail;
400 Detail.Documentation = Documentation;
401 Detail.CompletionDetail = CompletionDetail;
Eric Liuc5105f92018-02-16 14:15:55 +0000402 Detail.IncludeHeader = Include;
Sam McCall60039512018-02-09 14:42:01 +0000403 S.Detail = &Detail;
404
405 Symbols.insert(S);
406 return Symbols.find(S.ID);
407}
408
409void SymbolCollector::addDefinition(const NamedDecl &ND,
410 const Symbol &DeclSym) {
411 if (DeclSym.Definition)
412 return;
413 // If we saw some forward declaration, we end up copying the symbol.
414 // This is not ideal, but avoids duplicating the "is this a definition" check
415 // in clang::index. We should only see one definition.
416 Symbol S = DeclSym;
417 std::string FileURI;
418 if (auto DefLoc = getSymbolLocation(ND, ND.getASTContext().getSourceManager(),
Haojian Wudc02a3d2018-02-13 09:53:50 +0000419 Opts, ASTCtx->getLangOpts(), FileURI))
Sam McCall60039512018-02-09 14:42:01 +0000420 S.Definition = *DefLoc;
421 Symbols.insert(S);
422}
423
Haojian Wu4c1394d2017-12-12 15:42:10 +0000424} // namespace clangd
425} // namespace clang