Add a comment to explain why we can't just use getMethodFamily(),

as requested by Jordan's review. Swift SVN r30692
miichy · Jul 27, 2015 · a9742a2 · a9742a2
1 parent ccf21ff
commit a9742a2
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/lib/SIL/SILFunctionType.cpp b/lib/SIL/SILFunctionType.cpp
@@ -958,6 +958,10 @@ namespace {
       return ParameterConvention::Direct_Unowned;
     }
 
+    /// Given that a method returns a CF type, infer its method
+    /// family.  Unfortunately, Clang's getMethodFamily() never
+    /// considers a method to be in a special family if its result
+    /// doesn't satisfy isObjCRetainable().
     clang::ObjCMethodFamily getMethodFamilyForCFResult() const {
       // Trust an explicit attribute.
       if (auto attr = Method->getAttr<clang::ObjCMethodFamilyAttr>()) {