kadircet added a comment.

In D88414#2317106 <https://reviews.llvm.org/D88414#2317106>, @sammccall wrote:

> Now there's lots of usage (which looks good!) i'm finding it a bit hard to 
> keep track of what the tree will look like overall.
>
> At some point it'd be great to:
>  a) bind this to an LSP extension so we can see it in editors

i was also thinking about it and couldn't decide between a "custom command" vs 
"code action".

- the former gives a richer interaction, but requires every editor plugin to 
implement support.
- the latter is a little bit more restrictive but doesn't require a bunch of 
extra work.

I am happy to go with the "code action" approach initially. WDYT? (not in the 
scope of this patch)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88414

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

Reply via email to