diff options
| author | Fariborz Jahanian <fjahanian@apple.com> | 2011-01-03 18:08:02 +0000 |
|---|---|---|
| committer | Fariborz Jahanian <fjahanian@apple.com> | 2011-01-03 18:08:02 +0000 |
| commit | c63f1c5ab65b0318a0daf85bf02b936a07dd1921 (patch) | |
| tree | 0ecb0bf075ac3071ac0e8af19f075c380f7caafb | |
| parent | 7cd69249fa0d4b55da0731e451940caa17711af1 (diff) | |
| download | bcm5719-llvm-c63f1c5ab65b0318a0daf85bf02b936a07dd1921.tar.gz bcm5719-llvm-c63f1c5ab65b0318a0daf85bf02b936a07dd1921.zip | |
Guard lazy synthesis of provisional ivars under the new
-fobjc-default-synthesize-properties flag.
llvm-svn: 122757
| -rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 6 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaObjCProperty.cpp | 3 | ||||
| -rw-r--r-- | clang/test/SemaObjC/provisional-ivar-lookup.m | 2 |
3 files changed, 7 insertions, 4 deletions
diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index e481e5ce211..f0e20496469 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -1470,7 +1470,8 @@ ExprResult Sema::ActOnIdExpression(Scope *S, Expr *Ex = E.takeAs<Expr>(); if (Ex) return Owned(Ex); // Synthesize ivars lazily - if (getLangOptions().ObjCNonFragileABI2) { + if (getLangOptions().ObjCDefaultSynthProperties && + getLangOptions().ObjCNonFragileABI2) { if (SynthesizeProvisionalIvar(*this, R, II, NameLoc)) { if (const ObjCPropertyDecl *Property = canSynthesizeProvisionalIvar(II)) { @@ -1527,7 +1528,8 @@ ExprResult Sema::ActOnIdExpression(Scope *S, if (VarDecl *Var = R.getAsSingle<VarDecl>()) { if (getLangOptions().ObjCNonFragileABI && IvarLookupFollowUp && - !getLangOptions().ObjCNonFragileABI2 && + !(getLangOptions().ObjCDefaultSynthProperties && + getLangOptions().ObjCNonFragileABI2) && Var->isFileVarDecl()) { ObjCPropertyDecl *Property = canSynthesizeProvisionalIvar(II); if (Property) { diff --git a/clang/lib/Sema/SemaObjCProperty.cpp b/clang/lib/Sema/SemaObjCProperty.cpp index 65a447e2a1a..6aa1c693bc9 100644 --- a/clang/lib/Sema/SemaObjCProperty.cpp +++ b/clang/lib/Sema/SemaObjCProperty.cpp @@ -559,7 +559,8 @@ Decl *Sema::ActOnPropertyImplDecl(Scope *S, return 0; } IC->addPropertyImplementation(PIDecl); - if (getLangOptions().ObjCNonFragileABI2) { + if (getLangOptions().ObjCDefaultSynthProperties && + getLangOptions().ObjCNonFragileABI2) { // Diagnose if an ivar was lazily synthesdized due to a previous // use and if 1) property is @dynamic or 2) property is synthesized // but it requires an ivar of different name. diff --git a/clang/test/SemaObjC/provisional-ivar-lookup.m b/clang/test/SemaObjC/provisional-ivar-lookup.m index 526748538e0..6d57885d61f 100644 --- a/clang/test/SemaObjC/provisional-ivar-lookup.m +++ b/clang/test/SemaObjC/provisional-ivar-lookup.m @@ -1,4 +1,4 @@ -// RUN: %clang_cc1 -fsyntax-only -fobjc-nonfragile-abi2 -verify %s +// RUN: %clang_cc1 -fsyntax-only -fobjc-default-synthesize-properties -fobjc-nonfragile-abi2 -verify %s // rdar:// 8565343 @interface Foo { |
