annotate clang-tools-extra/clangd/Quality.cpp @ 221:79ff65ed7e25

LLVM12 Original
author Shinji KONO <kono@ie.u-ryukyu.ac.jp>
date Tue, 15 Jun 2021 19:15:29 +0900
parents 0572611fdcc8
children 5f17cb93ff66
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
150
anatofuz
parents:
diff changeset
1 //===--- Quality.cpp ---------------------------------------------*- C++-*-===//
anatofuz
parents:
diff changeset
2 //
anatofuz
parents:
diff changeset
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
anatofuz
parents:
diff changeset
4 // See https://llvm.org/LICENSE.txt for license information.
anatofuz
parents:
diff changeset
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
anatofuz
parents:
diff changeset
6 //
anatofuz
parents:
diff changeset
7 //===----------------------------------------------------------------------===//
anatofuz
parents:
diff changeset
8
anatofuz
parents:
diff changeset
9 #include "Quality.h"
anatofuz
parents:
diff changeset
10 #include "AST.h"
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
11 #include "CompletionModel.h"
150
anatofuz
parents:
diff changeset
12 #include "FileDistance.h"
anatofuz
parents:
diff changeset
13 #include "SourceCode.h"
anatofuz
parents:
diff changeset
14 #include "URI.h"
anatofuz
parents:
diff changeset
15 #include "index/Symbol.h"
anatofuz
parents:
diff changeset
16 #include "clang/AST/ASTContext.h"
anatofuz
parents:
diff changeset
17 #include "clang/AST/Decl.h"
anatofuz
parents:
diff changeset
18 #include "clang/AST/DeclCXX.h"
anatofuz
parents:
diff changeset
19 #include "clang/AST/DeclTemplate.h"
anatofuz
parents:
diff changeset
20 #include "clang/AST/DeclVisitor.h"
anatofuz
parents:
diff changeset
21 #include "clang/Basic/CharInfo.h"
anatofuz
parents:
diff changeset
22 #include "clang/Basic/SourceManager.h"
anatofuz
parents:
diff changeset
23 #include "clang/Sema/CodeCompleteConsumer.h"
anatofuz
parents:
diff changeset
24 #include "llvm/ADT/ArrayRef.h"
anatofuz
parents:
diff changeset
25 #include "llvm/ADT/SmallString.h"
anatofuz
parents:
diff changeset
26 #include "llvm/ADT/SmallVector.h"
anatofuz
parents:
diff changeset
27 #include "llvm/ADT/StringExtras.h"
anatofuz
parents:
diff changeset
28 #include "llvm/ADT/StringRef.h"
anatofuz
parents:
diff changeset
29 #include "llvm/Support/Casting.h"
anatofuz
parents:
diff changeset
30 #include "llvm/Support/FormatVariadic.h"
anatofuz
parents:
diff changeset
31 #include "llvm/Support/MathExtras.h"
anatofuz
parents:
diff changeset
32 #include "llvm/Support/raw_ostream.h"
anatofuz
parents:
diff changeset
33 #include <algorithm>
anatofuz
parents:
diff changeset
34 #include <cmath>
anatofuz
parents:
diff changeset
35
anatofuz
parents:
diff changeset
36 namespace clang {
anatofuz
parents:
diff changeset
37 namespace clangd {
anatofuz
parents:
diff changeset
38 static bool isReserved(llvm::StringRef Name) {
anatofuz
parents:
diff changeset
39 // FIXME: Should we exclude _Bool and others recognized by the standard?
anatofuz
parents:
diff changeset
40 return Name.size() >= 2 && Name[0] == '_' &&
anatofuz
parents:
diff changeset
41 (isUppercase(Name[1]) || Name[1] == '_');
anatofuz
parents:
diff changeset
42 }
anatofuz
parents:
diff changeset
43
anatofuz
parents:
diff changeset
44 static bool hasDeclInMainFile(const Decl &D) {
anatofuz
parents:
diff changeset
45 auto &SourceMgr = D.getASTContext().getSourceManager();
anatofuz
parents:
diff changeset
46 for (auto *Redecl : D.redecls()) {
anatofuz
parents:
diff changeset
47 if (isInsideMainFile(Redecl->getLocation(), SourceMgr))
anatofuz
parents:
diff changeset
48 return true;
anatofuz
parents:
diff changeset
49 }
anatofuz
parents:
diff changeset
50 return false;
anatofuz
parents:
diff changeset
51 }
anatofuz
parents:
diff changeset
52
anatofuz
parents:
diff changeset
53 static bool hasUsingDeclInMainFile(const CodeCompletionResult &R) {
anatofuz
parents:
diff changeset
54 const auto &Context = R.Declaration->getASTContext();
anatofuz
parents:
diff changeset
55 const auto &SourceMgr = Context.getSourceManager();
anatofuz
parents:
diff changeset
56 if (R.ShadowDecl) {
anatofuz
parents:
diff changeset
57 if (isInsideMainFile(R.ShadowDecl->getLocation(), SourceMgr))
anatofuz
parents:
diff changeset
58 return true;
anatofuz
parents:
diff changeset
59 }
anatofuz
parents:
diff changeset
60 return false;
anatofuz
parents:
diff changeset
61 }
anatofuz
parents:
diff changeset
62
anatofuz
parents:
diff changeset
63 static SymbolQualitySignals::SymbolCategory categorize(const NamedDecl &ND) {
anatofuz
parents:
diff changeset
64 if (const auto *FD = dyn_cast<FunctionDecl>(&ND)) {
anatofuz
parents:
diff changeset
65 if (FD->isOverloadedOperator())
anatofuz
parents:
diff changeset
66 return SymbolQualitySignals::Operator;
anatofuz
parents:
diff changeset
67 }
anatofuz
parents:
diff changeset
68 class Switch
anatofuz
parents:
diff changeset
69 : public ConstDeclVisitor<Switch, SymbolQualitySignals::SymbolCategory> {
anatofuz
parents:
diff changeset
70 public:
anatofuz
parents:
diff changeset
71 #define MAP(DeclType, Category) \
anatofuz
parents:
diff changeset
72 SymbolQualitySignals::SymbolCategory Visit##DeclType(const DeclType *) { \
anatofuz
parents:
diff changeset
73 return SymbolQualitySignals::Category; \
anatofuz
parents:
diff changeset
74 }
anatofuz
parents:
diff changeset
75 MAP(NamespaceDecl, Namespace);
anatofuz
parents:
diff changeset
76 MAP(NamespaceAliasDecl, Namespace);
anatofuz
parents:
diff changeset
77 MAP(TypeDecl, Type);
anatofuz
parents:
diff changeset
78 MAP(TypeAliasTemplateDecl, Type);
anatofuz
parents:
diff changeset
79 MAP(ClassTemplateDecl, Type);
anatofuz
parents:
diff changeset
80 MAP(CXXConstructorDecl, Constructor);
anatofuz
parents:
diff changeset
81 MAP(CXXDestructorDecl, Destructor);
anatofuz
parents:
diff changeset
82 MAP(ValueDecl, Variable);
anatofuz
parents:
diff changeset
83 MAP(VarTemplateDecl, Variable);
anatofuz
parents:
diff changeset
84 MAP(FunctionDecl, Function);
anatofuz
parents:
diff changeset
85 MAP(FunctionTemplateDecl, Function);
anatofuz
parents:
diff changeset
86 MAP(Decl, Unknown);
anatofuz
parents:
diff changeset
87 #undef MAP
anatofuz
parents:
diff changeset
88 };
anatofuz
parents:
diff changeset
89 return Switch().Visit(&ND);
anatofuz
parents:
diff changeset
90 }
anatofuz
parents:
diff changeset
91
anatofuz
parents:
diff changeset
92 static SymbolQualitySignals::SymbolCategory
anatofuz
parents:
diff changeset
93 categorize(const CodeCompletionResult &R) {
anatofuz
parents:
diff changeset
94 if (R.Declaration)
anatofuz
parents:
diff changeset
95 return categorize(*R.Declaration);
anatofuz
parents:
diff changeset
96 if (R.Kind == CodeCompletionResult::RK_Macro)
anatofuz
parents:
diff changeset
97 return SymbolQualitySignals::Macro;
anatofuz
parents:
diff changeset
98 // Everything else is a keyword or a pattern. Patterns are mostly keywords
anatofuz
parents:
diff changeset
99 // too, except a few which we recognize by cursor kind.
anatofuz
parents:
diff changeset
100 switch (R.CursorKind) {
anatofuz
parents:
diff changeset
101 case CXCursor_CXXMethod:
anatofuz
parents:
diff changeset
102 return SymbolQualitySignals::Function;
anatofuz
parents:
diff changeset
103 case CXCursor_ModuleImportDecl:
anatofuz
parents:
diff changeset
104 return SymbolQualitySignals::Namespace;
anatofuz
parents:
diff changeset
105 case CXCursor_MacroDefinition:
anatofuz
parents:
diff changeset
106 return SymbolQualitySignals::Macro;
anatofuz
parents:
diff changeset
107 case CXCursor_TypeRef:
anatofuz
parents:
diff changeset
108 return SymbolQualitySignals::Type;
anatofuz
parents:
diff changeset
109 case CXCursor_MemberRef:
anatofuz
parents:
diff changeset
110 return SymbolQualitySignals::Variable;
anatofuz
parents:
diff changeset
111 case CXCursor_Constructor:
anatofuz
parents:
diff changeset
112 return SymbolQualitySignals::Constructor;
anatofuz
parents:
diff changeset
113 default:
anatofuz
parents:
diff changeset
114 return SymbolQualitySignals::Keyword;
anatofuz
parents:
diff changeset
115 }
anatofuz
parents:
diff changeset
116 }
anatofuz
parents:
diff changeset
117
anatofuz
parents:
diff changeset
118 static SymbolQualitySignals::SymbolCategory
anatofuz
parents:
diff changeset
119 categorize(const index::SymbolInfo &D) {
anatofuz
parents:
diff changeset
120 switch (D.Kind) {
anatofuz
parents:
diff changeset
121 case index::SymbolKind::Namespace:
anatofuz
parents:
diff changeset
122 case index::SymbolKind::NamespaceAlias:
anatofuz
parents:
diff changeset
123 return SymbolQualitySignals::Namespace;
anatofuz
parents:
diff changeset
124 case index::SymbolKind::Macro:
anatofuz
parents:
diff changeset
125 return SymbolQualitySignals::Macro;
anatofuz
parents:
diff changeset
126 case index::SymbolKind::Enum:
anatofuz
parents:
diff changeset
127 case index::SymbolKind::Struct:
anatofuz
parents:
diff changeset
128 case index::SymbolKind::Class:
anatofuz
parents:
diff changeset
129 case index::SymbolKind::Protocol:
anatofuz
parents:
diff changeset
130 case index::SymbolKind::Extension:
anatofuz
parents:
diff changeset
131 case index::SymbolKind::Union:
anatofuz
parents:
diff changeset
132 case index::SymbolKind::TypeAlias:
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
133 case index::SymbolKind::TemplateTypeParm:
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
134 case index::SymbolKind::TemplateTemplateParm:
150
anatofuz
parents:
diff changeset
135 return SymbolQualitySignals::Type;
anatofuz
parents:
diff changeset
136 case index::SymbolKind::Function:
anatofuz
parents:
diff changeset
137 case index::SymbolKind::ClassMethod:
anatofuz
parents:
diff changeset
138 case index::SymbolKind::InstanceMethod:
anatofuz
parents:
diff changeset
139 case index::SymbolKind::StaticMethod:
anatofuz
parents:
diff changeset
140 case index::SymbolKind::InstanceProperty:
anatofuz
parents:
diff changeset
141 case index::SymbolKind::ClassProperty:
anatofuz
parents:
diff changeset
142 case index::SymbolKind::StaticProperty:
anatofuz
parents:
diff changeset
143 case index::SymbolKind::ConversionFunction:
anatofuz
parents:
diff changeset
144 return SymbolQualitySignals::Function;
anatofuz
parents:
diff changeset
145 case index::SymbolKind::Destructor:
anatofuz
parents:
diff changeset
146 return SymbolQualitySignals::Destructor;
anatofuz
parents:
diff changeset
147 case index::SymbolKind::Constructor:
anatofuz
parents:
diff changeset
148 return SymbolQualitySignals::Constructor;
anatofuz
parents:
diff changeset
149 case index::SymbolKind::Variable:
anatofuz
parents:
diff changeset
150 case index::SymbolKind::Field:
anatofuz
parents:
diff changeset
151 case index::SymbolKind::EnumConstant:
anatofuz
parents:
diff changeset
152 case index::SymbolKind::Parameter:
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
153 case index::SymbolKind::NonTypeTemplateParm:
150
anatofuz
parents:
diff changeset
154 return SymbolQualitySignals::Variable;
anatofuz
parents:
diff changeset
155 case index::SymbolKind::Using:
anatofuz
parents:
diff changeset
156 case index::SymbolKind::Module:
anatofuz
parents:
diff changeset
157 case index::SymbolKind::Unknown:
anatofuz
parents:
diff changeset
158 return SymbolQualitySignals::Unknown;
anatofuz
parents:
diff changeset
159 }
anatofuz
parents:
diff changeset
160 llvm_unreachable("Unknown index::SymbolKind");
anatofuz
parents:
diff changeset
161 }
anatofuz
parents:
diff changeset
162
anatofuz
parents:
diff changeset
163 static bool isInstanceMember(const NamedDecl *ND) {
anatofuz
parents:
diff changeset
164 if (!ND)
anatofuz
parents:
diff changeset
165 return false;
anatofuz
parents:
diff changeset
166 if (const auto *TP = dyn_cast<FunctionTemplateDecl>(ND))
anatofuz
parents:
diff changeset
167 ND = TP->TemplateDecl::getTemplatedDecl();
anatofuz
parents:
diff changeset
168 if (const auto *CM = dyn_cast<CXXMethodDecl>(ND))
anatofuz
parents:
diff changeset
169 return !CM->isStatic();
anatofuz
parents:
diff changeset
170 return isa<FieldDecl>(ND); // Note that static fields are VarDecl.
anatofuz
parents:
diff changeset
171 }
anatofuz
parents:
diff changeset
172
anatofuz
parents:
diff changeset
173 static bool isInstanceMember(const index::SymbolInfo &D) {
anatofuz
parents:
diff changeset
174 switch (D.Kind) {
anatofuz
parents:
diff changeset
175 case index::SymbolKind::InstanceMethod:
anatofuz
parents:
diff changeset
176 case index::SymbolKind::InstanceProperty:
anatofuz
parents:
diff changeset
177 case index::SymbolKind::Field:
anatofuz
parents:
diff changeset
178 return true;
anatofuz
parents:
diff changeset
179 default:
anatofuz
parents:
diff changeset
180 return false;
anatofuz
parents:
diff changeset
181 }
anatofuz
parents:
diff changeset
182 }
anatofuz
parents:
diff changeset
183
anatofuz
parents:
diff changeset
184 void SymbolQualitySignals::merge(const CodeCompletionResult &SemaCCResult) {
anatofuz
parents:
diff changeset
185 Deprecated |= (SemaCCResult.Availability == CXAvailability_Deprecated);
anatofuz
parents:
diff changeset
186 Category = categorize(SemaCCResult);
anatofuz
parents:
diff changeset
187
anatofuz
parents:
diff changeset
188 if (SemaCCResult.Declaration) {
anatofuz
parents:
diff changeset
189 ImplementationDetail |= isImplementationDetail(SemaCCResult.Declaration);
anatofuz
parents:
diff changeset
190 if (auto *ID = SemaCCResult.Declaration->getIdentifier())
anatofuz
parents:
diff changeset
191 ReservedName = ReservedName || isReserved(ID->getName());
anatofuz
parents:
diff changeset
192 } else if (SemaCCResult.Kind == CodeCompletionResult::RK_Macro)
anatofuz
parents:
diff changeset
193 ReservedName = ReservedName || isReserved(SemaCCResult.Macro->getName());
anatofuz
parents:
diff changeset
194 }
anatofuz
parents:
diff changeset
195
anatofuz
parents:
diff changeset
196 void SymbolQualitySignals::merge(const Symbol &IndexResult) {
anatofuz
parents:
diff changeset
197 Deprecated |= (IndexResult.Flags & Symbol::Deprecated);
anatofuz
parents:
diff changeset
198 ImplementationDetail |= (IndexResult.Flags & Symbol::ImplementationDetail);
anatofuz
parents:
diff changeset
199 References = std::max(IndexResult.References, References);
anatofuz
parents:
diff changeset
200 Category = categorize(IndexResult.SymInfo);
anatofuz
parents:
diff changeset
201 ReservedName = ReservedName || isReserved(IndexResult.Name);
anatofuz
parents:
diff changeset
202 }
anatofuz
parents:
diff changeset
203
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
204 float SymbolQualitySignals::evaluateHeuristics() const {
150
anatofuz
parents:
diff changeset
205 float Score = 1;
anatofuz
parents:
diff changeset
206
anatofuz
parents:
diff changeset
207 // This avoids a sharp gradient for tail symbols, and also neatly avoids the
anatofuz
parents:
diff changeset
208 // question of whether 0 references means a bad symbol or missing data.
anatofuz
parents:
diff changeset
209 if (References >= 10) {
anatofuz
parents:
diff changeset
210 // Use a sigmoid style boosting function, which flats out nicely for large
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
211 // numbers (e.g. 2.58 for 1M references).
150
anatofuz
parents:
diff changeset
212 // The following boosting function is equivalent to:
anatofuz
parents:
diff changeset
213 // m = 0.06
anatofuz
parents:
diff changeset
214 // f = 12.0
anatofuz
parents:
diff changeset
215 // boost = f * sigmoid(m * std::log(References)) - 0.5 * f + 0.59
anatofuz
parents:
diff changeset
216 // Sample data points: (10, 1.00), (100, 1.41), (1000, 1.82),
anatofuz
parents:
diff changeset
217 // (10K, 2.21), (100K, 2.58), (1M, 2.94)
anatofuz
parents:
diff changeset
218 float S = std::pow(References, -0.06);
anatofuz
parents:
diff changeset
219 Score *= 6.0 * (1 - S) / (1 + S) + 0.59;
anatofuz
parents:
diff changeset
220 }
anatofuz
parents:
diff changeset
221
anatofuz
parents:
diff changeset
222 if (Deprecated)
anatofuz
parents:
diff changeset
223 Score *= 0.1f;
anatofuz
parents:
diff changeset
224 if (ReservedName)
anatofuz
parents:
diff changeset
225 Score *= 0.1f;
anatofuz
parents:
diff changeset
226 if (ImplementationDetail)
anatofuz
parents:
diff changeset
227 Score *= 0.2f;
anatofuz
parents:
diff changeset
228
anatofuz
parents:
diff changeset
229 switch (Category) {
anatofuz
parents:
diff changeset
230 case Keyword: // Often relevant, but misses most signals.
anatofuz
parents:
diff changeset
231 Score *= 4; // FIXME: important keywords should have specific boosts.
anatofuz
parents:
diff changeset
232 break;
anatofuz
parents:
diff changeset
233 case Type:
anatofuz
parents:
diff changeset
234 case Function:
anatofuz
parents:
diff changeset
235 case Variable:
anatofuz
parents:
diff changeset
236 Score *= 1.1f;
anatofuz
parents:
diff changeset
237 break;
anatofuz
parents:
diff changeset
238 case Namespace:
anatofuz
parents:
diff changeset
239 Score *= 0.8f;
anatofuz
parents:
diff changeset
240 break;
anatofuz
parents:
diff changeset
241 case Macro:
anatofuz
parents:
diff changeset
242 case Destructor:
anatofuz
parents:
diff changeset
243 case Operator:
anatofuz
parents:
diff changeset
244 Score *= 0.5f;
anatofuz
parents:
diff changeset
245 break;
anatofuz
parents:
diff changeset
246 case Constructor: // No boost constructors so they are after class types.
anatofuz
parents:
diff changeset
247 case Unknown:
anatofuz
parents:
diff changeset
248 break;
anatofuz
parents:
diff changeset
249 }
anatofuz
parents:
diff changeset
250
anatofuz
parents:
diff changeset
251 return Score;
anatofuz
parents:
diff changeset
252 }
anatofuz
parents:
diff changeset
253
anatofuz
parents:
diff changeset
254 llvm::raw_ostream &operator<<(llvm::raw_ostream &OS,
anatofuz
parents:
diff changeset
255 const SymbolQualitySignals &S) {
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
256 OS << llvm::formatv("=== Symbol quality: {0}\n", S.evaluateHeuristics());
150
anatofuz
parents:
diff changeset
257 OS << llvm::formatv("\tReferences: {0}\n", S.References);
anatofuz
parents:
diff changeset
258 OS << llvm::formatv("\tDeprecated: {0}\n", S.Deprecated);
anatofuz
parents:
diff changeset
259 OS << llvm::formatv("\tReserved name: {0}\n", S.ReservedName);
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
260 OS << llvm::formatv("\tImplementation detail: {0}\n", S.ImplementationDetail);
150
anatofuz
parents:
diff changeset
261 OS << llvm::formatv("\tCategory: {0}\n", static_cast<int>(S.Category));
anatofuz
parents:
diff changeset
262 return OS;
anatofuz
parents:
diff changeset
263 }
anatofuz
parents:
diff changeset
264
anatofuz
parents:
diff changeset
265 static SymbolRelevanceSignals::AccessibleScope
anatofuz
parents:
diff changeset
266 computeScope(const NamedDecl *D) {
anatofuz
parents:
diff changeset
267 // Injected "Foo" within the class "Foo" has file scope, not class scope.
anatofuz
parents:
diff changeset
268 const DeclContext *DC = D->getDeclContext();
anatofuz
parents:
diff changeset
269 if (auto *R = dyn_cast_or_null<RecordDecl>(D))
anatofuz
parents:
diff changeset
270 if (R->isInjectedClassName())
anatofuz
parents:
diff changeset
271 DC = DC->getParent();
anatofuz
parents:
diff changeset
272 // Class constructor should have the same scope as the class.
anatofuz
parents:
diff changeset
273 if (isa<CXXConstructorDecl>(D))
anatofuz
parents:
diff changeset
274 DC = DC->getParent();
anatofuz
parents:
diff changeset
275 bool InClass = false;
anatofuz
parents:
diff changeset
276 for (; !DC->isFileContext(); DC = DC->getParent()) {
anatofuz
parents:
diff changeset
277 if (DC->isFunctionOrMethod())
anatofuz
parents:
diff changeset
278 return SymbolRelevanceSignals::FunctionScope;
anatofuz
parents:
diff changeset
279 InClass = InClass || DC->isRecord();
anatofuz
parents:
diff changeset
280 }
anatofuz
parents:
diff changeset
281 if (InClass)
anatofuz
parents:
diff changeset
282 return SymbolRelevanceSignals::ClassScope;
anatofuz
parents:
diff changeset
283 // ExternalLinkage threshold could be tweaked, e.g. module-visible as global.
anatofuz
parents:
diff changeset
284 // Avoid caching linkage if it may change after enclosing code completion.
anatofuz
parents:
diff changeset
285 if (hasUnstableLinkage(D) || D->getLinkageInternal() < ExternalLinkage)
anatofuz
parents:
diff changeset
286 return SymbolRelevanceSignals::FileScope;
anatofuz
parents:
diff changeset
287 return SymbolRelevanceSignals::GlobalScope;
anatofuz
parents:
diff changeset
288 }
anatofuz
parents:
diff changeset
289
anatofuz
parents:
diff changeset
290 void SymbolRelevanceSignals::merge(const Symbol &IndexResult) {
anatofuz
parents:
diff changeset
291 SymbolURI = IndexResult.CanonicalDeclaration.FileURI;
anatofuz
parents:
diff changeset
292 SymbolScope = IndexResult.Scope;
anatofuz
parents:
diff changeset
293 IsInstanceMember |= isInstanceMember(IndexResult.SymInfo);
anatofuz
parents:
diff changeset
294 if (!(IndexResult.Flags & Symbol::VisibleOutsideFile)) {
anatofuz
parents:
diff changeset
295 Scope = AccessibleScope::FileScope;
anatofuz
parents:
diff changeset
296 }
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
297 if (MainFileSignals) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
298 MainFileRefs =
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
299 std::max(MainFileRefs,
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
300 MainFileSignals->ReferencedSymbols.lookup(IndexResult.ID));
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
301 ScopeRefsInFile =
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
302 std::max(ScopeRefsInFile,
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
303 MainFileSignals->RelatedNamespaces.lookup(IndexResult.Scope));
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
304 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
305 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
306
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
307 void SymbolRelevanceSignals::computeASTSignals(
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
308 const CodeCompletionResult &SemaResult) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
309 if (!MainFileSignals)
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
310 return;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
311 if ((SemaResult.Kind != CodeCompletionResult::RK_Declaration) &&
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
312 (SemaResult.Kind != CodeCompletionResult::RK_Pattern))
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
313 return;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
314 if (const NamedDecl *ND = SemaResult.getDeclaration()) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
315 auto ID = getSymbolID(ND);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
316 if (!ID)
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
317 return;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
318 MainFileRefs =
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
319 std::max(MainFileRefs, MainFileSignals->ReferencedSymbols.lookup(ID));
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
320 if (const auto *NSD = dyn_cast<NamespaceDecl>(ND->getDeclContext())) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
321 if (NSD->isAnonymousNamespace())
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
322 return;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
323 std::string Scope = printNamespaceScope(*NSD);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
324 if (!Scope.empty())
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
325 ScopeRefsInFile = std::max(
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
326 ScopeRefsInFile, MainFileSignals->RelatedNamespaces.lookup(Scope));
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
327 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
328 }
150
anatofuz
parents:
diff changeset
329 }
anatofuz
parents:
diff changeset
330
anatofuz
parents:
diff changeset
331 void SymbolRelevanceSignals::merge(const CodeCompletionResult &SemaCCResult) {
anatofuz
parents:
diff changeset
332 if (SemaCCResult.Availability == CXAvailability_NotAvailable ||
anatofuz
parents:
diff changeset
333 SemaCCResult.Availability == CXAvailability_NotAccessible)
anatofuz
parents:
diff changeset
334 Forbidden = true;
anatofuz
parents:
diff changeset
335
anatofuz
parents:
diff changeset
336 if (SemaCCResult.Declaration) {
anatofuz
parents:
diff changeset
337 SemaSaysInScope = true;
anatofuz
parents:
diff changeset
338 // We boost things that have decls in the main file. We give a fixed score
anatofuz
parents:
diff changeset
339 // for all other declarations in sema as they are already included in the
anatofuz
parents:
diff changeset
340 // translation unit.
anatofuz
parents:
diff changeset
341 float DeclProximity = (hasDeclInMainFile(*SemaCCResult.Declaration) ||
anatofuz
parents:
diff changeset
342 hasUsingDeclInMainFile(SemaCCResult))
anatofuz
parents:
diff changeset
343 ? 1.0
anatofuz
parents:
diff changeset
344 : 0.6;
anatofuz
parents:
diff changeset
345 SemaFileProximityScore = std::max(DeclProximity, SemaFileProximityScore);
anatofuz
parents:
diff changeset
346 IsInstanceMember |= isInstanceMember(SemaCCResult.Declaration);
anatofuz
parents:
diff changeset
347 InBaseClass |= SemaCCResult.InBaseClass;
anatofuz
parents:
diff changeset
348 }
anatofuz
parents:
diff changeset
349
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
350 computeASTSignals(SemaCCResult);
150
anatofuz
parents:
diff changeset
351 // Declarations are scoped, others (like macros) are assumed global.
anatofuz
parents:
diff changeset
352 if (SemaCCResult.Declaration)
anatofuz
parents:
diff changeset
353 Scope = std::min(Scope, computeScope(SemaCCResult.Declaration));
anatofuz
parents:
diff changeset
354
anatofuz
parents:
diff changeset
355 NeedsFixIts = !SemaCCResult.FixIts.empty();
anatofuz
parents:
diff changeset
356 }
anatofuz
parents:
diff changeset
357
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
358 static float fileProximityScore(unsigned FileDistance) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
359 // Range: [0, 1]
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
360 // FileDistance = [0, 1, 2, 3, 4, .., FileDistance::Unreachable]
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
361 // Score = [1, 0.82, 0.67, 0.55, 0.45, .., 0]
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
362 if (FileDistance == FileDistance::Unreachable)
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
363 return 0;
150
anatofuz
parents:
diff changeset
364 // Assume approximately default options are used for sensible scoring.
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
365 return std::exp(FileDistance * -0.4f / FileDistanceOptions().UpCost);
150
anatofuz
parents:
diff changeset
366 }
anatofuz
parents:
diff changeset
367
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
368 static float scopeProximityScore(unsigned ScopeDistance) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
369 // Range: [0.6, 2].
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
370 // ScopeDistance = [0, 1, 2, 3, 4, 5, 6, 7, .., FileDistance::Unreachable]
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
371 // Score = [2.0, 1.55, 1.2, 0.93, 0.72, 0.65, 0.65, 0.65, .., 0.6]
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
372 if (ScopeDistance == FileDistance::Unreachable)
150
anatofuz
parents:
diff changeset
373 return 0.6f;
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
374 return std::max(0.65, 2.0 * std::pow(0.6, ScopeDistance / 2.0));
150
anatofuz
parents:
diff changeset
375 }
anatofuz
parents:
diff changeset
376
anatofuz
parents:
diff changeset
377 static llvm::Optional<llvm::StringRef>
anatofuz
parents:
diff changeset
378 wordMatching(llvm::StringRef Name, const llvm::StringSet<> *ContextWords) {
anatofuz
parents:
diff changeset
379 if (ContextWords)
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
380 for (const auto &Word : ContextWords->keys())
150
anatofuz
parents:
diff changeset
381 if (Name.contains_lower(Word))
anatofuz
parents:
diff changeset
382 return Word;
anatofuz
parents:
diff changeset
383 return llvm::None;
anatofuz
parents:
diff changeset
384 }
anatofuz
parents:
diff changeset
385
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
386 SymbolRelevanceSignals::DerivedSignals
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
387 SymbolRelevanceSignals::calculateDerivedSignals() const {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
388 DerivedSignals Derived;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
389 Derived.NameMatchesContext = wordMatching(Name, ContextWords).hasValue();
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
390 Derived.FileProximityDistance = !FileProximityMatch || SymbolURI.empty()
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
391 ? FileDistance::Unreachable
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
392 : FileProximityMatch->distance(SymbolURI);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
393 if (ScopeProximityMatch) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
394 // For global symbol, the distance is 0.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
395 Derived.ScopeProximityDistance =
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
396 SymbolScope ? ScopeProximityMatch->distance(*SymbolScope) : 0;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
397 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
398 return Derived;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
399 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
400
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
401 float SymbolRelevanceSignals::evaluateHeuristics() const {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
402 DerivedSignals Derived = calculateDerivedSignals();
150
anatofuz
parents:
diff changeset
403 float Score = 1;
anatofuz
parents:
diff changeset
404
anatofuz
parents:
diff changeset
405 if (Forbidden)
anatofuz
parents:
diff changeset
406 return 0;
anatofuz
parents:
diff changeset
407
anatofuz
parents:
diff changeset
408 Score *= NameMatch;
anatofuz
parents:
diff changeset
409
anatofuz
parents:
diff changeset
410 // File proximity scores are [0,1] and we translate them into a multiplier in
anatofuz
parents:
diff changeset
411 // the range from 1 to 3.
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
412 Score *= 1 + 2 * std::max(fileProximityScore(Derived.FileProximityDistance),
150
anatofuz
parents:
diff changeset
413 SemaFileProximityScore);
anatofuz
parents:
diff changeset
414
anatofuz
parents:
diff changeset
415 if (ScopeProximityMatch)
anatofuz
parents:
diff changeset
416 // Use a constant scope boost for sema results, as scopes of sema results
anatofuz
parents:
diff changeset
417 // can be tricky (e.g. class/function scope). Set to the max boost as we
anatofuz
parents:
diff changeset
418 // don't load top-level symbols from the preamble and sema results are
anatofuz
parents:
diff changeset
419 // always in the accessible scope.
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
420 Score *= SemaSaysInScope
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
421 ? 2.0
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
422 : scopeProximityScore(Derived.ScopeProximityDistance);
150
anatofuz
parents:
diff changeset
423
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
424 if (Derived.NameMatchesContext)
150
anatofuz
parents:
diff changeset
425 Score *= 1.5;
anatofuz
parents:
diff changeset
426
anatofuz
parents:
diff changeset
427 // Symbols like local variables may only be referenced within their scope.
anatofuz
parents:
diff changeset
428 // Conversely if we're in that scope, it's likely we'll reference them.
anatofuz
parents:
diff changeset
429 if (Query == CodeComplete) {
anatofuz
parents:
diff changeset
430 // The narrower the scope where a symbol is visible, the more likely it is
anatofuz
parents:
diff changeset
431 // to be relevant when it is available.
anatofuz
parents:
diff changeset
432 switch (Scope) {
anatofuz
parents:
diff changeset
433 case GlobalScope:
anatofuz
parents:
diff changeset
434 break;
anatofuz
parents:
diff changeset
435 case FileScope:
anatofuz
parents:
diff changeset
436 Score *= 1.5f;
anatofuz
parents:
diff changeset
437 break;
anatofuz
parents:
diff changeset
438 case ClassScope:
anatofuz
parents:
diff changeset
439 Score *= 2;
anatofuz
parents:
diff changeset
440 break;
anatofuz
parents:
diff changeset
441 case FunctionScope:
anatofuz
parents:
diff changeset
442 Score *= 4;
anatofuz
parents:
diff changeset
443 break;
anatofuz
parents:
diff changeset
444 }
anatofuz
parents:
diff changeset
445 } else {
anatofuz
parents:
diff changeset
446 // For non-completion queries, the wider the scope where a symbol is
anatofuz
parents:
diff changeset
447 // visible, the more likely it is to be relevant.
anatofuz
parents:
diff changeset
448 switch (Scope) {
anatofuz
parents:
diff changeset
449 case GlobalScope:
anatofuz
parents:
diff changeset
450 break;
anatofuz
parents:
diff changeset
451 case FileScope:
anatofuz
parents:
diff changeset
452 Score *= 0.5f;
anatofuz
parents:
diff changeset
453 break;
anatofuz
parents:
diff changeset
454 default:
anatofuz
parents:
diff changeset
455 // TODO: Handle other scopes as we start to use them for index results.
anatofuz
parents:
diff changeset
456 break;
anatofuz
parents:
diff changeset
457 }
anatofuz
parents:
diff changeset
458 }
anatofuz
parents:
diff changeset
459
anatofuz
parents:
diff changeset
460 if (TypeMatchesPreferred)
anatofuz
parents:
diff changeset
461 Score *= 5.0;
anatofuz
parents:
diff changeset
462
anatofuz
parents:
diff changeset
463 // Penalize non-instance members when they are accessed via a class instance.
anatofuz
parents:
diff changeset
464 if (!IsInstanceMember &&
anatofuz
parents:
diff changeset
465 (Context == CodeCompletionContext::CCC_DotMemberAccess ||
anatofuz
parents:
diff changeset
466 Context == CodeCompletionContext::CCC_ArrowMemberAccess)) {
anatofuz
parents:
diff changeset
467 Score *= 0.2f;
anatofuz
parents:
diff changeset
468 }
anatofuz
parents:
diff changeset
469
anatofuz
parents:
diff changeset
470 if (InBaseClass)
anatofuz
parents:
diff changeset
471 Score *= 0.5f;
anatofuz
parents:
diff changeset
472
anatofuz
parents:
diff changeset
473 // Penalize for FixIts.
anatofuz
parents:
diff changeset
474 if (NeedsFixIts)
anatofuz
parents:
diff changeset
475 Score *= 0.5f;
anatofuz
parents:
diff changeset
476
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
477 // Use a sigmoid style boosting function similar to `References`, which flats
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
478 // out nicely for large values. This avoids a sharp gradient for heavily
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
479 // referenced symbols. Use smaller gradient for ScopeRefsInFile since ideally
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
480 // MainFileRefs <= ScopeRefsInFile.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
481 if (MainFileRefs >= 2) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
482 // E.g.: (2, 1.12), (9, 2.0), (48, 3.0).
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
483 float S = std::pow(MainFileRefs, -0.11);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
484 Score *= 11.0 * (1 - S) / (1 + S) + 0.7;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
485 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
486 if (ScopeRefsInFile >= 2) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
487 // E.g.: (2, 1.04), (14, 2.0), (109, 3.0), (400, 3.6).
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
488 float S = std::pow(ScopeRefsInFile, -0.10);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
489 Score *= 10.0 * (1 - S) / (1 + S) + 0.7;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
490 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
491
150
anatofuz
parents:
diff changeset
492 return Score;
anatofuz
parents:
diff changeset
493 }
anatofuz
parents:
diff changeset
494
anatofuz
parents:
diff changeset
495 llvm::raw_ostream &operator<<(llvm::raw_ostream &OS,
anatofuz
parents:
diff changeset
496 const SymbolRelevanceSignals &S) {
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
497 OS << llvm::formatv("=== Symbol relevance: {0}\n", S.evaluateHeuristics());
150
anatofuz
parents:
diff changeset
498 OS << llvm::formatv("\tName: {0}\n", S.Name);
anatofuz
parents:
diff changeset
499 OS << llvm::formatv("\tName match: {0}\n", S.NameMatch);
anatofuz
parents:
diff changeset
500 if (S.ContextWords)
anatofuz
parents:
diff changeset
501 OS << llvm::formatv(
anatofuz
parents:
diff changeset
502 "\tMatching context word: {0}\n",
anatofuz
parents:
diff changeset
503 wordMatching(S.Name, S.ContextWords).getValueOr("<none>"));
anatofuz
parents:
diff changeset
504 OS << llvm::formatv("\tForbidden: {0}\n", S.Forbidden);
anatofuz
parents:
diff changeset
505 OS << llvm::formatv("\tNeedsFixIts: {0}\n", S.NeedsFixIts);
anatofuz
parents:
diff changeset
506 OS << llvm::formatv("\tIsInstanceMember: {0}\n", S.IsInstanceMember);
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
507 OS << llvm::formatv("\tInBaseClass: {0}\n", S.InBaseClass);
150
anatofuz
parents:
diff changeset
508 OS << llvm::formatv("\tContext: {0}\n", getCompletionKindString(S.Context));
anatofuz
parents:
diff changeset
509 OS << llvm::formatv("\tQuery type: {0}\n", static_cast<int>(S.Query));
anatofuz
parents:
diff changeset
510 OS << llvm::formatv("\tScope: {0}\n", static_cast<int>(S.Scope));
anatofuz
parents:
diff changeset
511
anatofuz
parents:
diff changeset
512 OS << llvm::formatv("\tSymbol URI: {0}\n", S.SymbolURI);
anatofuz
parents:
diff changeset
513 OS << llvm::formatv("\tSymbol scope: {0}\n",
anatofuz
parents:
diff changeset
514 S.SymbolScope ? *S.SymbolScope : "<None>");
anatofuz
parents:
diff changeset
515
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
516 SymbolRelevanceSignals::DerivedSignals Derived = S.calculateDerivedSignals();
150
anatofuz
parents:
diff changeset
517 if (S.FileProximityMatch) {
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
518 unsigned Score = fileProximityScore(Derived.FileProximityDistance);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
519 OS << llvm::formatv("\tIndex URI proximity: {0} (distance={1})\n", Score,
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
520 Derived.FileProximityDistance);
150
anatofuz
parents:
diff changeset
521 }
anatofuz
parents:
diff changeset
522 OS << llvm::formatv("\tSema file proximity: {0}\n", S.SemaFileProximityScore);
anatofuz
parents:
diff changeset
523
anatofuz
parents:
diff changeset
524 OS << llvm::formatv("\tSema says in scope: {0}\n", S.SemaSaysInScope);
anatofuz
parents:
diff changeset
525 if (S.ScopeProximityMatch)
anatofuz
parents:
diff changeset
526 OS << llvm::formatv("\tIndex scope boost: {0}\n",
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
527 scopeProximityScore(Derived.ScopeProximityDistance));
150
anatofuz
parents:
diff changeset
528
anatofuz
parents:
diff changeset
529 OS << llvm::formatv(
anatofuz
parents:
diff changeset
530 "\tType matched preferred: {0} (Context type: {1}, Symbol type: {2}\n",
anatofuz
parents:
diff changeset
531 S.TypeMatchesPreferred, S.HadContextType, S.HadSymbolType);
anatofuz
parents:
diff changeset
532
anatofuz
parents:
diff changeset
533 return OS;
anatofuz
parents:
diff changeset
534 }
anatofuz
parents:
diff changeset
535
anatofuz
parents:
diff changeset
536 float evaluateSymbolAndRelevance(float SymbolQuality, float SymbolRelevance) {
anatofuz
parents:
diff changeset
537 return SymbolQuality * SymbolRelevance;
anatofuz
parents:
diff changeset
538 }
anatofuz
parents:
diff changeset
539
221
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
540 DecisionForestScores
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
541 evaluateDecisionForest(const SymbolQualitySignals &Quality,
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
542 const SymbolRelevanceSignals &Relevance, float Base) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
543 Example E;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
544 E.setIsDeprecated(Quality.Deprecated);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
545 E.setIsReservedName(Quality.ReservedName);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
546 E.setIsImplementationDetail(Quality.ImplementationDetail);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
547 E.setNumReferences(Quality.References);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
548 E.setSymbolCategory(Quality.Category);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
549
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
550 SymbolRelevanceSignals::DerivedSignals Derived =
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
551 Relevance.calculateDerivedSignals();
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
552 int NumMatch = 0;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
553 if (Relevance.ContextWords) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
554 for (const auto &Word : Relevance.ContextWords->keys()) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
555 if (Relevance.Name.contains_lower(Word)) {
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
556 ++NumMatch;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
557 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
558 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
559 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
560 E.setIsNameInContext(NumMatch > 0);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
561 E.setNumNameInContext(NumMatch);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
562 E.setFractionNameInContext(
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
563 Relevance.ContextWords && !Relevance.ContextWords->empty()
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
564 ? NumMatch * 1.0 / Relevance.ContextWords->size()
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
565 : 0);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
566 E.setIsInBaseClass(Relevance.InBaseClass);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
567 E.setFileProximityDistanceCost(Derived.FileProximityDistance);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
568 E.setSemaFileProximityScore(Relevance.SemaFileProximityScore);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
569 E.setSymbolScopeDistanceCost(Derived.ScopeProximityDistance);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
570 E.setSemaSaysInScope(Relevance.SemaSaysInScope);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
571 E.setScope(Relevance.Scope);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
572 E.setContextKind(Relevance.Context);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
573 E.setIsInstanceMember(Relevance.IsInstanceMember);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
574 E.setHadContextType(Relevance.HadContextType);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
575 E.setHadSymbolType(Relevance.HadSymbolType);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
576 E.setTypeMatchesPreferred(Relevance.TypeMatchesPreferred);
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
577
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
578 DecisionForestScores Scores;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
579 // Exponentiating DecisionForest prediction makes the score of each tree a
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
580 // multiplciative boost (like NameMatch). This allows us to weigh the
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
581 // prediciton score and NameMatch appropriately.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
582 Scores.ExcludingName = pow(Base, Evaluate(E));
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
583 // Following cases are not part of the generated training dataset:
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
584 // - Symbols with `NeedsFixIts`.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
585 // - Forbidden symbols.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
586 // - Keywords: Dataset contains only macros and decls.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
587 if (Relevance.NeedsFixIts)
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
588 Scores.ExcludingName *= 0.5;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
589 if (Relevance.Forbidden)
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
590 Scores.ExcludingName *= 0;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
591 if (Quality.Category == SymbolQualitySignals::Keyword)
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
592 Scores.ExcludingName *= 4;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
593
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
594 // NameMatch should be a multiplier on total score to support rescoring.
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
595 Scores.Total = Relevance.NameMatch * Scores.ExcludingName;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
596 return Scores;
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
597 }
79ff65ed7e25 LLVM12 Original
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 173
diff changeset
598
150
anatofuz
parents:
diff changeset
599 // Produces an integer that sorts in the same order as F.
anatofuz
parents:
diff changeset
600 // That is: a < b <==> encodeFloat(a) < encodeFloat(b).
anatofuz
parents:
diff changeset
601 static uint32_t encodeFloat(float F) {
anatofuz
parents:
diff changeset
602 static_assert(std::numeric_limits<float>::is_iec559, "");
anatofuz
parents:
diff changeset
603 constexpr uint32_t TopBit = ~(~uint32_t{0} >> 1);
anatofuz
parents:
diff changeset
604
anatofuz
parents:
diff changeset
605 // Get the bits of the float. Endianness is the same as for integers.
anatofuz
parents:
diff changeset
606 uint32_t U = llvm::FloatToBits(F);
anatofuz
parents:
diff changeset
607 // IEEE 754 floats compare like sign-magnitude integers.
anatofuz
parents:
diff changeset
608 if (U & TopBit) // Negative float.
anatofuz
parents:
diff changeset
609 return 0 - U; // Map onto the low half of integers, order reversed.
anatofuz
parents:
diff changeset
610 return U + TopBit; // Positive floats map onto the high half of integers.
anatofuz
parents:
diff changeset
611 }
anatofuz
parents:
diff changeset
612
anatofuz
parents:
diff changeset
613 std::string sortText(float Score, llvm::StringRef Name) {
anatofuz
parents:
diff changeset
614 // We convert -Score to an integer, and hex-encode for readability.
anatofuz
parents:
diff changeset
615 // Example: [0.5, "foo"] -> "41000000foo"
anatofuz
parents:
diff changeset
616 std::string S;
anatofuz
parents:
diff changeset
617 llvm::raw_string_ostream OS(S);
anatofuz
parents:
diff changeset
618 llvm::write_hex(OS, encodeFloat(-Score), llvm::HexPrintStyle::Lower,
anatofuz
parents:
diff changeset
619 /*Width=*/2 * sizeof(Score));
anatofuz
parents:
diff changeset
620 OS << Name;
anatofuz
parents:
diff changeset
621 OS.flush();
anatofuz
parents:
diff changeset
622 return S;
anatofuz
parents:
diff changeset
623 }
anatofuz
parents:
diff changeset
624
anatofuz
parents:
diff changeset
625 llvm::raw_ostream &operator<<(llvm::raw_ostream &OS,
anatofuz
parents:
diff changeset
626 const SignatureQualitySignals &S) {
anatofuz
parents:
diff changeset
627 OS << llvm::formatv("=== Signature Quality:\n");
anatofuz
parents:
diff changeset
628 OS << llvm::formatv("\tNumber of parameters: {0}\n", S.NumberOfParameters);
anatofuz
parents:
diff changeset
629 OS << llvm::formatv("\tNumber of optional parameters: {0}\n",
anatofuz
parents:
diff changeset
630 S.NumberOfOptionalParameters);
anatofuz
parents:
diff changeset
631 OS << llvm::formatv("\tKind: {0}\n", S.Kind);
anatofuz
parents:
diff changeset
632 return OS;
anatofuz
parents:
diff changeset
633 }
anatofuz
parents:
diff changeset
634
anatofuz
parents:
diff changeset
635 } // namespace clangd
anatofuz
parents:
diff changeset
636 } // namespace clang