chenwj added a comment.

> 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?


https://reviews.llvm.org/D32759



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

Reply via email to