tree 4b1aa8ec5389973f59f974d1eb53b913be9bd1c2
parent 09fc2fc1ebc030552e46047608907d10aaed5cce
author Adam Czachorowski <adamcz@google.com> 1633539013 +0200
committer Copybara-Service <copybara-worker@google.com> 1637247954 -0800

[clang][clangd] Improve signature help for variadic functions.

This covers both C-style variadic functions and template variadic w/
parameter packs.

Previously we would return no signatures when working with template
variadic functions once activeParameter reached the position of the
parameter pack (except when it was the only param, then we'd still
show it when no arguments were given). With this commit, we now show
signathure help correctly.

Additionally, this commit fixes the activeParameter value in LSP output
of clangd in the presence of variadic functions (both kinds). LSP does
not allow the activeParamter to be higher than the number of parameters
in the active signature. With "..." or parameter pack being just one
argument, for all but first argument passed to "..." we'd report
incorrect activeParameter value. Clients such as VSCode would then treat
it as 0, as suggested in the spec) and highlight the wrong parameter.

In the future, we should add support for per-signature activeParamter
value, which exists in LSP since 3.16.0. This is not part of this
commit.

Differential Revision: https://reviews.llvm.org/D111318

GitOrigin-RevId: 55a79318c60d8a39329195f43bf43b89da9a638e