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

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

Reply via email to