hokein added inline comments.
================
Comment at: clang-tools-extra/clangd/index/MemIndex.h:72
// A map from (subject, predicate) pair to objects.
- llvm::DenseMap<std::pair<SymbolID, index::SymbolRole>, std::vector<SymbolID>>
+ llvm::DenseMap<std::pair<SymbolID, RelationKind>, std::vector<SymbolID>>
Relations;
----------------
kadircet wrote:
> can we rather use `uint8_t` in here instead of `RelationKind` to get rid of
> the `DenseMapInfo` specialization completely?
Yes, done.
================
Comment at: clang-tools-extra/clangd/index/Relation.h:22
+enum class RelationKind : uint8_t {
+ BaseOf,
----------------
kadircet wrote:
> is this clang-formatted ?
yes, note that I'm leaving a trailing "," on purpose, to prevent clang-format
the whole structure into one-line.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68981/new/
https://reviews.llvm.org/D68981
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
