kadircet added a comment.

> I thought the point of the comment was us not handling it properly rather 
> than LSP not supporting it (e.g. LSP does support Operator but we do not). 
> Then, the comment about ctor and dtor being indistinguishable probably 
> belongs to Protocol.h/cpp and SymbolKind there in particular?

If that was the case I would expect this comment to be above previous line, 
i.e. `index::getSymbolInfo`, as that's the one doing the clang-level mapping. 
Whereas `indexSymbolKindToSymbolKind` does LSP mapping.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89935/new/

https://reviews.llvm.org/D89935

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to