Enhance reference-count checker to correctly identify CG "release" functions. This...

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@57997 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Analysis/CFRefCount.cpp b/lib/Analysis/CFRefCount.cpp
index acf4b568ede18a3b85c720bd288cf631be3a6410..db8a4598ba366c463d8691cea7cd879d3fac5c92 100644 (file)
--- a/lib/Analysis/CFRefCount.cpp
+++ b/lib/Analysis/CFRefCount.cpp
@@ -704,8 +704,12 @@ RetainSummary* RetainSummaryManager::getSummary(FunctionDecl* FD) {
       }
     }
 
-    if (FName[0] == 'C' && FName[1] == 'F')
-      S = getCFSummary(FD, FName);  
+    if (FName[0] == 'C') {
+      if (FName[1] == 'F')
+        S = getCFSummary(FD, FName);  
+      else if (FName[1] == 'G')
+        S = getCGSummary(FD, FName);
+    }
     else if (FName[0] == 'N' && FName[1] == 'S')
       S = getNSSummary(FD, FName);
   }
@@ -834,10 +838,14 @@ RetainSummary* RetainSummaryManager::getCFSummaryCreateRule(FunctionDecl* FD) {
  
   FunctionType* FT =
     dyn_cast<FunctionType>(FD->getType().getTypePtr());
-  
-  if (FT && !isCFRefType(FT->getResultType()))
-    return getPersistentSummary(RetEffect::MakeNoRet());
 
+  if (FT) {
+    QualType ResTy = FT->getResultType();
+    
+    if (!isCFRefType(ResTy) && !isCGRefType(ResTy))
+      return getPersistentSummary(RetEffect::MakeNoRet());
+  }
+  
   assert (ScratchArgs.empty());
   
   if (FD->getIdentifier() == CFDictionaryCreateII) {

diff --git a/test/Analysis/CGColorSpace.c b/test/Analysis/CGColorSpace.c
new file mode 100644 (file)
index 0000000..793f75f
--- /dev/null
+++ b/test/Analysis/CGColorSpace.c
@@ -0,0 +1,18 @@
+// clang -checker-cfref -verify %s
+
+typedef struct CGColorSpace *CGColorSpaceRef;
+extern CGColorSpaceRef CGColorSpaceCreateDeviceRGB(void);
+extern CGColorSpaceRef CGColorSpaceRetain(CGColorSpaceRef space);
+extern void CGColorSpaceRelease(CGColorSpaceRef space);
+
+void f() {
+  CGColorSpaceRef X = CGColorSpaceCreateDeviceRGB();
+  CGColorSpaceRetain(X);  // expected-warning{{leak}}
+}
+
+void fb() {
+  CGColorSpaceRef X = CGColorSpaceCreateDeviceRGB();
+  CGColorSpaceRetain(X);
+  CGColorSpaceRelease(X);
+  CGColorSpaceRelease(X);  // no-warning
+}