[PATCH] D51605: [clangd] SymbolOccurrences -> Refs and cleanup

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added inline comments.


================
Comment at: clangd/index/Index.cpp:158
+    auto &SymRefs = Sym.second;
+    std::sort(SymRefs.begin(), SymRefs.end());
+    // TODO: do we really need to dedup?
----------------
I noticed this by accident, but i'm pretty sure `std::sort()` should not be used in LLVM sources **at all**.
`llvm::sort()` should be used **everywhere**.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51605