]> granicus.if.org Git - clang/commitdiff
fix rdar://8445858 - __sync_* intrinsics erroneously reject ObjC pointers
authorChris Lattner <sabre@nondot.org>
Fri, 17 Sep 2010 21:12:38 +0000 (21:12 +0000)
committerChris Lattner <sabre@nondot.org>
Fri, 17 Sep 2010 21:12:38 +0000 (21:12 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@114209 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaChecking.cpp
test/SemaObjC/exprs.m

index 0ed2f33e0e20e988ed99c964b884beaf8110f94b..790e7671ed942a5c71eacde2704e15e226515237 100644 (file)
@@ -427,7 +427,7 @@ Sema::SemaBuiltinAtomicOverloaded(ExprResult TheCallResult) {
 
   QualType ValType =
     FirstArg->getType()->getAs<PointerType>()->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();
index 3370bda0517af8dcacf7fbf633aff86e6d01a15f..13c34e5650de94044e0d93a23e1ba8e9d4987d67 100644 (file)
@@ -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);
+}