steveire added a comment. In D56444#1350935 <https://reviews.llvm.org/D56444#1350935>, @sammccall wrote:
> In D56444#1350897 <https://reviews.llvm.org/D56444#1350897>, @steveire wrote: > > > > > > I'm not inclined to hold off fixing this bug though, because: > > - it's blocking other patches, and the changes you're talking about seem > likely to be quite involved and need some details worked out > - if we are going to switch from the RAV parent map to an > ASTDumper-traverser-derived map, then the closer that switch is to a no-op, > the easier it will be to land. That makes sense to me, but I wanted to draw your attention to the refactoring effort. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56444/new/ https://reviews.llvm.org/D56444 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
