diff options
| author | Fangrui Song <maskray@google.com> | 2018-10-07 17:21:08 +0000 |
|---|---|---|
| committer | Fangrui Song <maskray@google.com> | 2018-10-07 17:21:08 +0000 |
| commit | 8380c9e918c2b1be0840fe67bb394419ec41ea83 (patch) | |
| tree | 4cbf7a2edf6415c44bdcab971eb89abda4638497 | |
| parent | ecc8af61e78bce8075475a947538f7599aa0587e (diff) | |
| download | bcm5719-llvm-8380c9e918c2b1be0840fe67bb394419ec41ea83.tar.gz bcm5719-llvm-8380c9e918c2b1be0840fe67bb394419ec41ea83.zip | |
[clangd] Migrate to LLVM STLExtras range API
llvm-svn: 343946
| -rw-r--r-- | clang-tools-extra/clangd/FileDistance.cpp | 4 | ||||
| -rw-r--r-- | clang-tools-extra/clangd/XRefs.cpp | 9 |
2 files changed, 6 insertions, 7 deletions
diff --git a/clang-tools-extra/clangd/FileDistance.cpp b/clang-tools-extra/clangd/FileDistance.cpp index e24d030dd84..e530ad5235d 100644 --- a/clang-tools-extra/clangd/FileDistance.cpp +++ b/clang-tools-extra/clangd/FileDistance.cpp @@ -72,8 +72,8 @@ FileDistance::FileDistance(StringMap<SourceParams> Sources, Rest = parent_path(Rest, sys::path::Style::posix); auto NextHash = hash_value(Rest); auto &Down = DownEdges[NextHash]; - if (std::find(Down.begin(), Down.end(), Hash) == Down.end()) - DownEdges[NextHash].push_back(Hash); + if (!llvm::is_contained(Down, Hash)) + Down.push_back(Hash); // We can't just break after MaxUpTraversals, must still set DownEdges. if (I > S.getValue().MaxUpTraversals) { if (Cache.find(Hash) != Cache.end()) diff --git a/clang-tools-extra/clangd/XRefs.cpp b/clang-tools-extra/clangd/XRefs.cpp index 7d9635e9c23..4c718220676 100644 --- a/clang-tools-extra/clangd/XRefs.cpp +++ b/clang-tools-extra/clangd/XRefs.cpp @@ -372,11 +372,10 @@ public: } std::vector<Reference> take() && { - std::sort(References.begin(), References.end(), - [](const Reference &L, const Reference &R) { - return std::tie(L.Loc, L.CanonicalTarget, L.Role) < - std::tie(R.Loc, R.CanonicalTarget, R.Role); - }); + llvm::sort(References, [](const Reference &L, const Reference &R) { + return std::tie(L.Loc, L.CanonicalTarget, L.Role) < + std::tie(R.Loc, R.CanonicalTarget, R.Role); + }); // We sometimes see duplicates when parts of the AST get traversed twice. References.erase( std::unique(References.begin(), References.end(), |

