Allow accessing 'isa' via '->' operator.

(fixes radar 7447251).

llvm-svn: 90795

2 files changed, 10 insertions, 3 deletions

diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp
index 6c5a1ec05c5..b3c5c9fc20b 100644
--- a/clang/lib/Sema/SemaExpr.cpp
+++ b/clang/lib/Sema/SemaExpr.cpp
@@ -2522,6 +2522,16 @@ Sema::LookupMemberExpr(LookupResult &R, Expr *&BaseExpr,
   // If this is an Objective-C pseudo-builtin and a definition is provided then
   // use that.
   if (BaseType->isObjCIdType()) {
+    if (IsArrow) {
+      // Handle the following exceptional case PObj->isa.
+      if (const ObjCObjectPointerType *OPT =
+          BaseType->getAs<ObjCObjectPointerType>()) {
+        if (OPT->getPointeeType()->isSpecificBuiltinType(BuiltinType::ObjCId) &&
+            MemberName.getAsIdentifierInfo()->isStr("isa"))
+          return Owned(new (Context) ObjCIsaExpr(BaseExpr, false, MemberLoc,
+                                                 Context.getObjCIdType()));
+      }
+    }
     // We have an 'id' type. Rather than fall through, we check if this
     // is a reference to 'isa'.
     if (BaseType != Context.ObjCIdRedefinitionType) {
diff --git a/clang/test/SemaObjC/id-isa-ref.m b/clang/test/SemaObjC/id-isa-ref.m
index fa3293ce79b..dbb6b2f53d8 100644
--- a/clang/test/SemaObjC/id-isa-ref.m
+++ b/clang/test/SemaObjC/id-isa-ref.m
@@ -1,8 +1,5 @@
 // RUN: clang-cc -fsyntax-only -verify %s
 
-// Failing currently due to Obj-C type representation changes. 2009-09-17
-// XFAIL: *
-
 typedef struct objc_object {
   struct objc_class *isa;
 } *id;