[PATCH] D131295: [clangd] Implement textDocument/codeLens

WangWei via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 17 18:40:03 PST 2022


lightmelodies added a comment.

In D131295#3935027 <https://reviews.llvm.org/D131295#3935027>, @Trass3r wrote:

> Thanks, it seems to fix the base case, but I still see multiple lenses when I add
>
>   template <int V>
>   int Foo<V>::foo()
>   {
>       return 0;
>   }

Interesting, I see the same issue with DocumentSymbols. Maybe because **foo** is explicit specialization (though outside **Foo<1>**) so both check are passed. However I can not find a way to detect such case right now.
F25364530: image.png <https://reviews.llvm.org/F25364530>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131295



More information about the cfe-commits mailing list