malaperle added a comment.
Hi! Have you looked into https://reviews.llvm.org/D40548 ? Maybe we need to
coordinate the two a bit.
================
Comment at: clangd/Symbol.h:37
+// The class presents a C++ symbol, e.g. class, function.
+struct Symbol {
+ // The symbol identifier, using USR.
----------------
I think it would be nice to have methods as an interface to get this data
instead of storing them directly. So that an index-on-disk could go fetch the
data. Especially the occurrences which can take a lot of memory (I'm working on
a branch that does that). But perhaps defining that interface is not within the
scope of this patch and could be better discussed in D40548 ?
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D40897
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
