[PATCH] D119077: clangd SemanticHighlighting: added support for highlighting overloaded operators

[Iannis de Zwart via Phabricator via cfe-commits]

iannisdezwart added a comment.

Thank you for the comment, it's very helpful.

I didn't know exactly how to get all operator token locations and `getCXXOperatorNameRange()`.
This is going to be extremely helpful and will clean up the code to make it more readable.
Thank you for the detailed list of how to get the full operator name token location for each specific token type.

About your comment on the implicit calls to `operator[]` and `operator()`,
this was also a workaround. Do you have any idea how to get both parentheses from a single `DeclRefExpr`?
I researched this, but haven't yet found the answer.

I will try incorporating these changes today or tomorrow, if I can find time.


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

https://reviews.llvm.org/D119077