[PATCH] D32759: Fix errored return value in CheckFunctionReturnType and add a fixit hint

Wei-Ren Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 3 06:11:43 PDT 2017


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





More information about the cfe-commits mailing list