sammccall added a comment.

In D72494#1813875 <https://reviews.llvm.org/D72494#1813875>, @sammccall wrote:

> In D72494#1813719 <https://reviews.llvm.org/D72494#1813719>, @ilya-biryukov 
> wrote:
>
> > Could you also add a test (and possibly support this in the visitors) for 
> > `findExplicitReferences`?
>
>
> I've written tests, but it doesn't work and it's not obvious to me yet how to 
> make it work, so I'll do this in a separate patch.


By which I mean, adding the same delegation to the RAV there doesn't work, and 
I'm not confident that jiggling it around until the test passes is correct 
here, so I want to understand the traversal behavior a bit better first (RAV 
has several special cases around OVE/POE)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72494



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

Reply via email to