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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
