sammccall added a comment.

In D135506#3844919 <https://reviews.llvm.org/D135506#3844919>, @tom-anders 
wrote:

> lgtm, thanks! With this change in place, we could probably ajdust 
> `pickDeclToUse` in Hover.cpp (introduced in https://reviews.llvm.org/D133664) 
> to also check for `UsingDecl` instead of `BaseUsingDecl`, right?

Yes, I think so!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135506

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

Reply via email to