sammccall marked 4 inline comments as done. sammccall added inline comments.
================ Comment at: clangd/CodeComplete.cpp:245 + // Methods are simply grouped by name. + return hash_combine('M', IndexResult->Name); + case index::SymbolKind::Function: ---------------- ilya-biryukov wrote: > Members never come from the index for completion, right? Maybe add an assert > here instead? Done. I'd keep the actual return here as it's simple and explains the value. We likely want to use it in future to augment members with signals from the index. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47957 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits