diff --git a/include/clang/Basic/DiagnosticSemaKinds.td b/include/clang/Basic/DiagnosticSemaKinds.td index 81a1f4a8498d964c9efff798ead1471e59e7cb64..76730e6fc7a3234ec73552cdb09eb659e29ffcf7 100644 --- a/include/clang/Basic/DiagnosticSemaKinds.td +++ b/include/clang/Basic/DiagnosticSemaKinds.td @@ -4561,9 +4561,6 @@ def err_synthesizing_arc_weak_property_disabled : Error< def err_synthesizing_arc_weak_property_no_runtime : Error< "cannot synthesize weak property because the current deployment target " "does not support weak references">; -def warn_objc_weak_compat : Warning< - "the meaning of __weak has changed in manual reference-counting">, - InGroup<DiagGroup<"objc-weak-compat">>, DefaultIgnore; def err_arc_unsupported_weak_class : Error< "class is incompatible with __weak references">; def err_arc_weak_unavailable_assign : Error< diff --git a/lib/Sema/SemaType.cpp b/lib/Sema/SemaType.cpp index bbd0c3fbd0668c9e162757deb18200e30eea5dd7..b6bc15942829c8b3a9c1ee2da3d4b92d30c529ba 100644 --- a/lib/Sema/SemaType.cpp +++ b/lib/Sema/SemaType.cpp @@ -5187,6 +5187,18 @@ static bool handleObjCOwnershipTypeAttr(TypeProcessingState &state, type = S.Context.getAttributedType(AttributedType::attr_objc_ownership, origType, type); + auto diagnoseOrDelay = [](Sema &S, SourceLocation loc, + unsigned diagnostic, QualType type) { + if (S.DelayedDiagnostics.shouldDelayDiagnostics()) { + S.DelayedDiagnostics.add( + sema::DelayedDiagnostic::makeForbiddenType( + S.getSourceManager().getExpansionLoc(loc), + diagnostic, type, /*ignored*/ 0)); + } else { + S.Diag(loc, diagnostic); + } + }; + // Sometimes, __weak isn't allowed. if (lifetime == Qualifiers::OCL_Weak && !S.getLangOpts().ObjCWeak && !NonObjCPointer) { @@ -5197,26 +5209,12 @@ static bool handleObjCOwnershipTypeAttr(TypeProcessingState &state, : diag::err_arc_weak_no_runtime); // In any case, delay the diagnostic until we know what we're parsing. - if (S.DelayedDiagnostics.shouldDelayDiagnostics()) { - S.DelayedDiagnostics.add( - sema::DelayedDiagnostic::makeForbiddenType( - S.getSourceManager().getExpansionLoc(AttrLoc), - diagnostic, type, /*ignored*/ 0)); - } else { - S.Diag(AttrLoc, diagnostic); - } + diagnoseOrDelay(S, AttrLoc, diagnostic, type); attr.setInvalid(); return true; } - // If we accepted __weak, we might still need to warn about it. - if (lifetime == Qualifiers::OCL_Weak && - !S.getLangOpts().ObjCAutoRefCount && - S.getLangOpts().ObjCWeak) { - S.Diag(AttrLoc, diag::warn_objc_weak_compat); - } - // Forbid __weak for class objects marked as // objc_arc_weak_reference_unavailable if (lifetime == Qualifiers::OCL_Weak) { diff --git a/test/SemaObjC/property-in-class-extension-1.m b/test/SemaObjC/property-in-class-extension-1.m index 8f5907b54d321a6df44ba03db3ce40c094fffd12..b25639cf06247f987bdc99c6d73f59b81f85282d 100644 --- a/test/SemaObjC/property-in-class-extension-1.m +++ b/test/SemaObjC/property-in-class-extension-1.m @@ -1,5 +1,5 @@ -// RUN: %clang_cc1 -fsyntax-only -triple x86_64-apple-darwin11 -fobjc-runtime-has-weak -fobjc-weak -verify -Weverything -Wno-objc-weak-compat %s -// RUN: %clang_cc1 -x objective-c++ -triple x86_64-apple-darwin11 -fobjc-runtime-has-weak -fobjc-weak -fsyntax-only -verify -Weverything -Wno-objc-weak-compat %s +// RUN: %clang_cc1 -fsyntax-only -triple x86_64-apple-darwin11 -fobjc-runtime-has-weak -fobjc-weak -verify -Weverything %s +// RUN: %clang_cc1 -x objective-c++ -triple x86_64-apple-darwin11 -fobjc-runtime-has-weak -fobjc-weak -fsyntax-only -verify -Weverything %s // rdar://12103400 @class NSString;