[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 11:15:27 PDT 2017


erichkeane updated this revision to Diff 98495.
erichkeane added a reviewer: eli.friedman.
erichkeane added a comment.

Added a test, found a slight difference in behavior!


https://reviews.llvm.org/D32759

Files:
  lib/Sema/SemaType.cpp
  test/SemaObjC/method-bad-param.m


Index: lib/Sema/SemaType.cpp
===================================================================
--- lib/Sema/SemaType.cpp
+++ lib/Sema/SemaType.cpp
@@ -2285,8 +2285,9 @@
   // Methods cannot return interface types. All ObjC objects are
   // passed by reference.
   if (T->isObjCObjectType()) {
-    Diag(Loc, diag::err_object_cannot_be_passed_returned_by_value) << 0 << T;
-    return 0;
+    Diag(Loc, diag::err_object_cannot_be_passed_returned_by_value)
+        << 0 << T << FixItHint::CreateInsertion(Loc, "*");
+    return true;
   }
 
   return false;
Index: test/SemaObjC/method-bad-param.m
===================================================================
--- test/SemaObjC/method-bad-param.m
+++ test/SemaObjC/method-bad-param.m
@@ -20,6 +20,12 @@
 }
 @end
 
+// Ensure that this function is properly marked as a failure.
+void func_with_bad_call(bar* b, foo* f) {
+  [b cccccc:5]; // expected-warning {{instance method '-cccccc:' not found}}
+                // expected-note at -17 {{receiver is instance of class declared here}}
+}
+
 void somefunc(foo x) {} // expected-error {{interface type 'foo' cannot be passed by value; did you forget * in 'foo'}}
 foo somefunc2() {} // expected-error {{interface type 'foo' cannot be returned by value; did you forget * in 'foo'}}
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32759.98495.patch
Type: text/x-patch
Size: 1288 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170510/95dc6003/attachment-0001.bin>


More information about the cfe-commits mailing list