erichkeane added a comment. In https://reviews.llvm.org/D32759#744613, @chenwj wrote:
> > Ah, sorry. "Our Tests" means the lit test SemaObjC/method-bad-param.m > > (line 11). I ran the lit tests initially with a breakpoint on this line > > and it never hit, though I must have set up the debugger wrong. Once I > > replaced it with an assert, method-bad-param failed. > > The fix LGTM. Is it possible to tweak method-bad-param.m so that we can see > the difference after the fix? I actually couldn't come up with a way where we COULD see the difference... I was hoiping someone else could come up with something if it were important enough for us. https://reviews.llvm.org/D32759 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
