diff options
| author | Douglas Gregor <dgregor@apple.com> | 2012-04-13 15:53:08 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2012-04-13 15:53:08 +0000 |
| commit | eba02fecdd3063960091de39853b07225d03381d (patch) | |
| tree | 28307d62869039acba3a1c29ef9de79feb8af24a | |
| parent | 20735d2df89e7a6f9df1ded1cb534566ec34cd28 (diff) | |
| download | bcm5719-llvm-eba02fecdd3063960091de39853b07225d03381d.tar.gz bcm5719-llvm-eba02fecdd3063960091de39853b07225d03381d.zip | |
super and class property reference expressions don't need to be
rebuilt. Fixes <rdar://problem/11052352>.
llvm-svn: 154667
| -rw-r--r-- | clang/lib/Sema/SemaPseudoObject.cpp | 9 | ||||
| -rw-r--r-- | clang/test/SemaObjCXX/property-reference.mm | 15 |
2 files changed, 22 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaPseudoObject.cpp b/clang/lib/Sema/SemaPseudoObject.cpp index d52c912457a..3b2cf8d7350 100644 --- a/clang/lib/Sema/SemaPseudoObject.cpp +++ b/clang/lib/Sema/SemaPseudoObject.cpp @@ -1300,8 +1300,13 @@ static Expr *stripOpaqueValuesFromPseudoObjectRef(Sema &S, Expr *E) { Expr *opaqueRef = E->IgnoreParens(); if (ObjCPropertyRefExpr *refExpr = dyn_cast<ObjCPropertyRefExpr>(opaqueRef)) { - OpaqueValueExpr *baseOVE = cast<OpaqueValueExpr>(refExpr->getBase()); - return ObjCPropertyRefRebuilder(S, baseOVE->getSourceExpr()).rebuild(E); + if (refExpr->isObjectReceiver()) { + OpaqueValueExpr *baseOVE = cast<OpaqueValueExpr>(refExpr->getBase()); + return ObjCPropertyRefRebuilder(S, baseOVE->getSourceExpr()).rebuild(E); + } + + // Neither class or super property references need a rebuild. + return E; } else if (ObjCSubscriptRefExpr *refExpr = dyn_cast<ObjCSubscriptRefExpr>(opaqueRef)) { OpaqueValueExpr *baseOVE = cast<OpaqueValueExpr>(refExpr->getBaseExpr()); diff --git a/clang/test/SemaObjCXX/property-reference.mm b/clang/test/SemaObjCXX/property-reference.mm index 18f06045a34..b86ae5e9f51 100644 --- a/clang/test/SemaObjCXX/property-reference.mm +++ b/clang/test/SemaObjCXX/property-reference.mm @@ -42,3 +42,18 @@ typedef const TCPPObject& CREF_TCPPObject; cppObjectNonAtomic = cppObject; } @end + + +// <rdar://problem/11052352> +@interface NSObject ++ alloc; +- init; +- class; +@end + +template<typename T> void f() { + NSObject *o = [NSObject.alloc init]; + [o class]; +} + +template void f<int>(); |

