[PATCH] D143260: [clangd] Add semantic token for labels

Christian Kandeler via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 14 04:41:38 PDT 2023


ckandeler added a comment.

In my opinion, it is not possible to have a competitive client if you limit yourself to the official LSP feature set; you just need language-specific extensions in practice.
And while of course not every silly idea should be blindly accepted, I think this "lowest common denominator" approach artificially limits clients' potential feature sets and inhibits innovation.
I am of course biased, but it seems to me that when in doubt, client requirements should take precedence over server-side purity concerns.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143260



More information about the cfe-commits mailing list