[clang-tools-extra] [clangd] Extract to function: Pass unmodified scalar parameters by value (PR #227675)

[Christian Kandeler via cfe-commits]

ckandeler wrote:

> I also found that we have a problem with function pointers
> 
> ```c++
>  void foo(void (*F)(int)) {
>       [[F(1);]]
>  }
> ```
> 
> Extract to function leads to incorrect extracted function parameter type`void extracted(void (*)(int) F)`.
> 
> But maybe we can fix this separately, because this was not introduced by this PR.

Yes, seems out of scope here. I have addressed all the other issues.

https://github.com/llvm/llvm-project/pull/227675