diff options
| author | Sam McCall <sam.mccall@gmail.com> | 2017-03-09 10:47:44 +0000 |
|---|---|---|
| committer | Sam McCall <sam.mccall@gmail.com> | 2017-03-09 10:47:44 +0000 |
| commit | 573050e703308c624e48c7f36f19dc2d83d7170f (patch) | |
| tree | 117d50c89d6e50bda3d7fdd100a747702017e037 /clang-tools-extra/include-fixer/IncludeFixer.cpp | |
| parent | 76630d43f6081c657ac5be2406061d43334045dd (diff) | |
| download | bcm5719-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.cpp | 3 |
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); } |

