[PATCH] D130265: [clangd] resolve forwarded parameters in Hover

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 8 02:58:35 PDT 2022


kadircet added a comment.

This looks like a nice idea, but i think we definitely need annotations to prevent confusion. I'd prefer something like:
assume `foo(int x, Args... args);`

  function foo
  -> void
  Parameters:
  - int x
  - int a : forwarded to [bar:a](it'd be great if we could have location links here)
  - int b : forwarded to [bar:b](again location links)

There's https://github.com/microsoft/language-server-protocol/issues/379 for asking to support links in hover, maybe we just need a way to point at a range inside a file


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130265



More information about the cfe-commits mailing list