From: Chris Lattner Date: Fri, 17 Sep 2010 21:12:38 +0000 (+0000) Subject: fix rdar://8445858 - __sync_* intrinsics erroneously reject ObjC pointers X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dd5fa7a21ca9db9d620001ca977681a1b724e3fe;p=clang fix rdar://8445858 - __sync_* intrinsics erroneously reject ObjC pointers git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@114209 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Sema/SemaChecking.cpp b/lib/Sema/SemaChecking.cpp index 0ed2f33e0e..790e7671ed 100644 --- a/lib/Sema/SemaChecking.cpp +++ b/lib/Sema/SemaChecking.cpp @@ -427,7 +427,7 @@ Sema::SemaBuiltinAtomicOverloaded(ExprResult TheCallResult) { QualType ValType = FirstArg->getType()->getAs()->getPointeeType(); - if (!ValType->isIntegerType() && !ValType->isPointerType() && + if (!ValType->isIntegerType() && !ValType->isAnyPointerType() && !ValType->isBlockPointerType()) { Diag(DRE->getLocStart(), diag::err_atomic_builtin_must_be_pointer_intptr) << FirstArg->getType() << FirstArg->getSourceRange(); diff --git a/test/SemaObjC/exprs.m b/test/SemaObjC/exprs.m index 3370bda051..13c34e5650 100644 --- a/test/SemaObjC/exprs.m +++ b/test/SemaObjC/exprs.m @@ -22,3 +22,13 @@ void test2() { #define MAX(A,B) ({ __typeof__(A) __a = (A); __typeof__(B) __b = (B); __a < __b ? __b : __a; }) void (^foo)(int, int) = ^(int x, int y) { int z = MAX(x, y); }; + + + +// rdar://8445858 +@class Object; +static Object *g; +void test3(Object *o) { + // this is ok. + __sync_bool_compare_and_swap(&g, 0, o); +}