[PATCH] D140775: [clangd] Hover: show CalleeArgInfo for literals
Nathan Ridge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 2 00:21:10 PST 2023
nridge added a comment.
Do you want to add a simple test case for a non-literal expression? Something like hovering over the `+` in `2 + 3` should work.
Also, this is pre-existing, but I noticed that in a case like this:
void bar(int);
void func() {
int x = 42;
bar(x);
}
the hover for `x` includes a line that just says `Passed` -- is that useful in any way? Should we just omit the CalleeArgInfo a case like that?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140775/new/
https://reviews.llvm.org/D140775
More information about the cfe-commits
mailing list