From: Ted Kremenek Date: Sun, 21 Feb 2010 05:15:32 +0000 (+0000) Subject: Clang really intends to reject attribute 'warn_unused_result' on Objective-C methods... X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=48108fdb9c3bcffe5bccd88a1a91353a4df71629;p=clang Clang really intends to reject attribute 'warn_unused_result' on Objective-C methods, but instead it crashes on them. We might extend this attribute to work on methods, but for now fix the crasher. Addresses . git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@96723 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Sema/SemaDeclAttr.cpp b/lib/Sema/SemaDeclAttr.cpp index ef82d53b5c..8178a14751 100644 --- a/lib/Sema/SemaDeclAttr.cpp +++ b/lib/Sema/SemaDeclAttr.cpp @@ -754,7 +754,7 @@ static void HandleWarnUnusedResult(Decl *D, const AttributeList &Attr, Sema &S) return; } - if (!isFunctionOrMethod(D)) { + if (!isFunction(D)) { S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type) << Attr.getName() << 0 /*function*/; return; diff --git a/test/SemaObjC/method-warn-unused-attribute.m b/test/SemaObjC/method-warn-unused-attribute.m new file mode 100644 index 0000000000..d9dcf996ec --- /dev/null +++ b/test/SemaObjC/method-warn-unused-attribute.m @@ -0,0 +1,8 @@ +// RUN: %clang_cc1 -fsyntax-only -Wunused-value -verify %s + +@interface INTF +// Currently this is rejected by both GCC and Clang (and Clang was crashing on it). +- (id) foo __attribute__((warn_unused_result)); // expected-warning{{warning: 'warn_unused_result' attribute only applies to function types}} +@end + +