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

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 10 12:24:46 PDT 2017


erichkeane added a comment.

In https://reviews.llvm.org/D32759#751339, @efriedma wrote:

> Our behavior for your testcase is less than ideal (we should be generating an invalid declaration instead of just dropping it on the floor), but that's not something you need to fix here.
>
> Could you also include the Objective-C++ testcase I wrote?  Or does it not work the way I expected?


I hadn't seen it.  I see from your diff now that you have a .mm file (p4.mm) which I presume is what you mean.  I'll check on it and get back to you on it.


https://reviews.llvm.org/D32759





More information about the cfe-commits mailing list