[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn added a comment. In D92041#2454236 , @sammccall wrote: > Very nice, thanks! > I'll land this for you now. Thank you very much! Learned a lot about clangd and clang AST from this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn updated this revision to Diff 311256. xndcn added a comment. Thank you, it works like a charm! For class withou template, `getHoverInfo(QualType ...)` will add namespace scope by default, so I have to add `SuppressScope` printpolicy here. CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn added a comment. In D92041#2446366 , @sammccall wrote: > Sorry for the delay here. Kadir is out on vacation. > > Yikes - it's a shame reusing our existing type printing doesn't do the right > thing, but injected-classname and partial specializations

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn updated this revision to Diff 308994. xndcn added a comment. `getHoverInfo(CXXThisExpr->getType()->getPointeeType(), ...)` does not output namespace scope and template parameters without specialization: F14499632: Pointee.png while `getHoverInfo(CXXThis

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn added a comment. Thanks, it look more clear really. I'm trying to make the hover looks like `auto` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92041/new/ https://reviews.llvm.org/D92041 ___ cfe-commits mailing list cfe-commits@lists.

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn added a comment. In D92041#2424869 , @kadircet wrote: > can you give me an email address to associate the commit with? xnd...@gmail.com, thank you. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92041/new/ https://reviews.llvm.org/D92041

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn added a comment. In D92041#2420251 , @kadircet wrote: > Do you have commit access or should I commit this for you? I still don't have commit access, so please help me commit this, thank you! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn updated this revision to Diff 307947. xndcn added a comment. Thanks. Update commit to fix the last nit. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92041/new/ https://reviews.llvm.org/D92041 Files: clang-tools-extra/clangd/Hover.cpp clang-tools-extra/clangd/unittests/HoverT

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn updated this revision to Diff 307749. xndcn added a comment. Thanks! Update the commit as review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92041/new/ https://reviews.llvm.org/D92041 Files: clang-tools-extra/clangd/Hover.cpp clang-tools-extra/clangd/unittests/Hov

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn updated this revision to Diff 307622. xndcn added a comment. Update the diff with `getHoverContents(const NamedDecl ..)` overload function. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92041/new/ https://reviews.llvm.org/D92041 Files: clang-tools-extra/clangd/Hover.cpp clang

[PATCH] D92041: [clangd] Add hover info for `this` expr

xndcn added inline comments. Comment at: clang-tools-extra/clangd/Hover.cpp:610 +/// Generate a \p Hover object given the \p this pointer. +HoverInfo getHoverContents(const CXXThisExpr *CTE, const SymbolIndex *Index) { + const NamedDecl *D = CTE->getType()->getPointeeType()->get

[PATCH] D92041: Add hover info for `this` expr

xndcn added inline comments. Comment at: clang-tools-extra/clangd/Hover.cpp:610 +/// Generate a \p Hover object given the \p this pointer. +HoverInfo getHoverContents(const CXXThisExpr *CTE, const SymbolIndex *Index) { + const NamedDecl *D = CTE->getType()->getPointeeType()->get

[PATCH] D92041: Add hover info for `this` expr

xndcn created this revision. xndcn added reviewers: sammccall, kadircet. xndcn added a project: clang-tools-extra. Herald added subscribers: cfe-commits, usaxena95, arphaman. Herald added a project: clang. xndcn requested review of this revision. How about add hover information for `this` expr? It