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

[WangWei via Phabricator via cfe-commits]

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