[cfe-commits] r91515 - in /cfe/trunk: include/clang/Basic/Builtins.def test/Analysis/security-syntax-checks.m test/SemaObjC/newproperty-class-method-1.m test/SemaObjC/property-13.m test/SemaObjC/property-user-setter.m

[Eli Friedman]

Author: efriedma
Date: Wed Dec 16 00:28:21 2009
New Revision: 91515

URL: http://llvm.org/viewvc/llvm-project?rev=91515&view=rev
Log:
Add abort() as a builtin.  This has two effects: one, we warn for incorrect 
declarations of abort(), and two, we mark it noreturn.  Missing the latter
shows up in one of the "embarassing" tests (from the thread on llvmdev
"detailed comparison of generated code size for LLVM and other compilers").


Modified:
    cfe/trunk/include/clang/Basic/Builtins.def
    cfe/trunk/test/Analysis/security-syntax-checks.m
    cfe/trunk/test/SemaObjC/newproperty-class-method-1.m
    cfe/trunk/test/SemaObjC/property-13.m
    cfe/trunk/test/SemaObjC/property-user-setter.m

Modified: cfe/trunk/include/clang/Basic/Builtins.def
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/Builtins.def?rev=91515&r1=91514&r2=91515&view=diff

==============================================================================
--- cfe/trunk/include/clang/Basic/Builtins.def (original)
+++ cfe/trunk/include/clang/Basic/Builtins.def Wed Dec 16 00:28:21 2009
@@ -475,6 +475,7 @@
 
 // C99 library functions
 // C99 stdlib.h
+LIBBUILTIN(abort, "v",            "fr",    "stdlib.h")
 LIBBUILTIN(calloc, "v*zz",        "f",     "stdlib.h")
 LIBBUILTIN(exit, "vi",            "fr",    "stdlib.h")
 LIBBUILTIN(_Exit, "vi",           "fr",    "stdlib.h")

Modified: cfe/trunk/test/Analysis/security-syntax-checks.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Analysis/security-syntax-checks.m?rev=91515&r1=91514&r2=91515&view=diff

==============================================================================
--- cfe/trunk/test/Analysis/security-syntax-checks.m (original)
+++ cfe/trunk/test/Analysis/security-syntax-checks.m Wed Dec 16 00:28:21 2009
@@ -48,6 +48,7 @@
 int setregid(gid_t, gid_t);
 int setreuid(uid_t, uid_t);
 extern void check(int);
+void abort(void);
 
 void test_setuid() 
 {

Modified: cfe/trunk/test/SemaObjC/newproperty-class-method-1.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaObjC/newproperty-class-method-1.m?rev=91515&r1=91514&r2=91515&view=diff

==============================================================================
--- cfe/trunk/test/SemaObjC/newproperty-class-method-1.m (original)
+++ cfe/trunk/test/SemaObjC/newproperty-class-method-1.m Wed Dec 16 00:28:21 2009
@@ -1,5 +1,7 @@
 // RUN: %clang_cc1 %s -verify -fsyntax-only
 
+void abort(void);
+
 @interface Subclass
 + (int)magicNumber;
 + (void)setMagicNumber:(int)value;

Modified: cfe/trunk/test/SemaObjC/property-13.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaObjC/property-13.m?rev=91515&r1=91514&r2=91515&view=diff

==============================================================================
--- cfe/trunk/test/SemaObjC/property-13.m (original)
+++ cfe/trunk/test/SemaObjC/property-13.m Wed Dec 16 00:28:21 2009
@@ -44,6 +44,7 @@
 - (void) setOptional_getter_attr:(int)value { ivar = value; }
 @end
 
+void abort(void);
 int main ()
 {
 	Test *x = [[Test alloc] init];

Modified: cfe/trunk/test/SemaObjC/property-user-setter.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaObjC/property-user-setter.m?rev=91515&r1=91514&r2=91515&view=diff

==============================================================================
--- cfe/trunk/test/SemaObjC/property-user-setter.m (original)
+++ cfe/trunk/test/SemaObjC/property-user-setter.m Wed Dec 16 00:28:21 2009
@@ -80,6 +80,7 @@
 }
 @end
 
+void abort(void);
 int main (void) {
     Subclass *x = [[Subclass alloc] init];