Added fixmes.

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

diff --git a/lib/Sema/SemaExprObjC.cpp b/lib/Sema/SemaExprObjC.cpp
index 108ff3c21e074aca58e02a06c9f93286c1168c3e..d833efe309ef05c3cc994688ec031e33785a725a 100644 (file)
--- a/lib/Sema/SemaExprObjC.cpp
+++ b/lib/Sema/SemaExprObjC.cpp
@@ -339,6 +339,8 @@ static bool ClassImplementsProtocol(ObjCProtocolDecl *lProto,
     // In gcc, it is also allowed assigning a protocol-qualified 'id'
     // type to a LHS object when protocol in qualified LHS is in list
     // of protocols in the rhs 'id' object. This IMO, should be a bug.
+    // FIXME: Treat this as an extension, and flag this as an error when
+    //  GCC extensions are not enabled.
     else if (RHSIsQualifiedID &&
              ProtocolCompatibleWithProtocol(protoList[i], lProto))
       return true;

diff --git a/test/Sema/objc-comptypes-9.m b/test/Sema/objc-comptypes-9.m
index 0358fc39d0ea3e00adcc4695ea5f5acc182a8ffd..4f0a277e74e1c6a7b697b54fe855199d58df5eec 100644 (file)
--- a/test/Sema/objc-comptypes-9.m
+++ b/test/Sema/objc-comptypes-9.m
@@ -1,4 +1,6 @@
 // RUN: clang -fsyntax-only %s
+// FIXME: This test case tests the patch applied in: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20080602/006017.html
+//   Eventually that logic should be treated as an extension.
 
 typedef signed char BOOL;
 typedef int NSInteger;