Thanks Jordan. I’ve committed the patch in r263818. I didn’t understand your comment on WeakObjectProfileTy’s table (I’m assuming you are talking about the table in ScopeInfo.h:183). It looks like the entry MyClass.prop in the table already covers the case this patch fixed (in the test case I added, Base is NSBundle and Property is the method “foo”)?
> On Mar 18, 2016, at 9:55 AM, Jordan Rose via cfe-commits > <cfe-commits@lists.llvm.org> wrote: > > jordan_rose accepted this revision. > jordan_rose added a comment. > This revision is now accepted and ready to land. > > Ah, of course! Thanks for catching this, Akira. Can you add this case to the > table in the doc comment for WeakObjectProfileTy? (That's how I convinced > myself it was correct.) > > > http://reviews.llvm.org/D18268 > > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
