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

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

Reply via email to