This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Log warning when using legacy (theia) semantic highlighting.
ClosedPublic

Authored by sammccall on Jan 20 2021, 3:39 AM.

Details

Summary

The legacy protocol will be removed on trunk after the 12 branch cut,
and gone in clangd 13.

Diff Detail

Event Timeline

sammccall created this revision.Jan 20 2021, 3:39 AM
sammccall requested review of this revision.Jan 20 2021, 3:39 AM
kadircet accepted this revision.Jan 20 2021, 4:57 AM
kadircet added a subscriber: nridge.

LGTM, a friendly ping to @nridge though, as he was working on theia related stuff (at least in the past, not sure if you still do :D), to give him a heads-up if this is going to require some changes on their end.

This revision is now accepted and ready to land.Jan 20 2021, 4:57 AM

Thanks for the heads up! It would be nice to get D77702 landed before the removal of TheiaSemanticHighlighting. I'll try to find some time to rebase it.

Thanks for the heads up! It would be nice to get D77702 landed before the removal of TheiaSemanticHighlighting. I'll try to find some time to rebase it.

Good point!
I've revived D77811, as I'd prefer not to start adding modifiers before can make them independent of kinds. (Moving features to different infrastructure tends to involve some backward-compatibility/testing toil...)

From there I think it should be easy to get the rest of this merged:

  • scope modifiers make a lot of sense and should be straightforward to add
  • dependent type vs name: maybe we should consider making dependent a modifier instead? but otherwise LG
  • the "member" issue I think has been resolved for the most part, by clarifying the spec