From 06769f919735a3d5d64ab45003f1c517c8b865ea Mon Sep 17 00:00:00 2001 From: Fariborz Jahanian Date: Tue, 2 Feb 2010 18:35:07 +0000 Subject: [PATCH] Fix up rewriting of protocol qualified types in objc rewriter. Fixes radar 7589414. llvm-svn: 95097 --- clang/lib/Frontend/RewriteObjC.cpp | 10 ++++++++- .../Rewriter/rewrite-protocol-qualified.mm | 21 +++++++++++++++++++ 2 files changed, 30 insertions(+), 1 deletion(-) create mode 100644 clang/test/Rewriter/rewrite-protocol-qualified.mm diff --git a/clang/lib/Frontend/RewriteObjC.cpp b/clang/lib/Frontend/RewriteObjC.cpp index df43da7efff3..980c96ad17d9 100644 --- a/clang/lib/Frontend/RewriteObjC.cpp +++ b/clang/lib/Frontend/RewriteObjC.cpp @@ -1995,7 +1995,15 @@ static void scanToNextArgument(const char *&argRef) { } bool RewriteObjC::needToScanForQualifiers(QualType T) { - return T->isObjCQualifiedIdType() || T->isObjCQualifiedInterfaceType(); + if (const PointerType *PT = T->getAs()) { + if (PT->getPointeeType()->isObjCQualifiedIdType()) + return true; + } + if (T->isObjCObjectPointerType()) { + T = T->getPointeeType(); + return T->isObjCQualifiedInterfaceType(); + } + return false; } void RewriteObjC::RewriteObjCQualifiedInterfaceTypes(Expr *E) { diff --git a/clang/test/Rewriter/rewrite-protocol-qualified.mm b/clang/test/Rewriter/rewrite-protocol-qualified.mm new file mode 100644 index 000000000000..e93ddbba2771 --- /dev/null +++ b/clang/test/Rewriter/rewrite-protocol-qualified.mm @@ -0,0 +1,21 @@ +// RUN: %clang_cc1 -x objective-c++ -fblocks -fms-extensions -rewrite-objc %s -o %t-rw.cpp +// RUN: FileCheck -check-prefix LP --input-file=%t-rw.cpp %s +// radar 7589414 + +@protocol NSPortDelegate; +@interface NSConnection @end + +@interface NSMessagePort +- (void) clone; +@end + +@implementation NSMessagePort +- (void) clone { + NSConnection *conn = 0; + id *idc = 0; +} +@end + +// CHECK-LP: NSConnection /**/ *conn = 0; + +// CHECK-LP: id /**/ *idc = 0;