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