summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/include-fixer/IncludeFixer.cpp
diff options
context:
space:
mode:
authorSam McCall <sam.mccall@gmail.com>2017-03-09 10:47:44 +0000
committerSam McCall <sam.mccall@gmail.com>2017-03-09 10:47:44 +0000
commit573050e703308c624e48c7f36f19dc2d83d7170f (patch)
tree117d50c89d6e50bda3d7fdd100a747702017e037 /clang-tools-extra/include-fixer/IncludeFixer.cpp
parent76630d43f6081c657ac5be2406061d43334045dd (diff)
downloadbcm5719-llvm-573050e703308c624e48c7f36f19dc2d83d7170f.tar.gz
bcm5719-llvm-573050e703308c624e48c7f36f19dc2d83d7170f.zip
[include-fixer] Remove line number from Symbol identity
Summary: Remove line number from Symbol identity. For our purposes (include-fixer and clangd autocomplete), function overloads within the same header should mostly be treated as a single combined symbol. We may want to track individual occurrences (line number, full type info) and aggregate this during mapreduce, but that's not done here. Reviewers: hokein, bkramer Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D30685 llvm-svn: 297371
Diffstat (limited to 'clang-tools-extra/include-fixer/IncludeFixer.cpp')
-rw-r--r--clang-tools-extra/include-fixer/IncludeFixer.cpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/clang-tools-extra/include-fixer/IncludeFixer.cpp b/clang-tools-extra/include-fixer/IncludeFixer.cpp
index ca4d863d3ad..e68231b8b5e 100644
--- a/clang-tools-extra/include-fixer/IncludeFixer.cpp
+++ b/clang-tools-extra/include-fixer/IncludeFixer.cpp
@@ -333,8 +333,7 @@ IncludeFixerContext IncludeFixerSemaSource::getIncludeFixerContext(
: "\"" + FilePath + "\""),
SourceManager, HeaderSearch);
SymbolCandidates.emplace_back(Symbol.getName(), Symbol.getSymbolKind(),
- MinimizedFilePath, Symbol.getLineNumber(),
- Symbol.getContexts());
+ MinimizedFilePath, Symbol.getContexts());
}
return IncludeFixerContext(FilePath, QuerySymbolInfos, SymbolCandidates);
}
OpenPOWER on IntegriCloud