Fixes a minor hick up to my last patch.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@142711 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Sema/SemaDeclObjC.cpp b/lib/Sema/SemaDeclObjC.cpp
index 69d9c1d..f3b7ad7 100644
--- a/lib/Sema/SemaDeclObjC.cpp
+++ b/lib/Sema/SemaDeclObjC.cpp
@@ -2321,11 +2321,15 @@
bool containsInvalidMethodImplAttribute(ObjCMethodDecl *IMD,
const AttrVec &A) {
// If method is only declared in implementation (private method),
- // or method declared in interface has no attribute.
// No need to issue any diagnostics on method definition with attributes.
- if (!IMD || !IMD->hasAttrs())
+ if (!IMD)
return false;
+ // method declared in interface has no attribute.
+ // But implementation has attributes. This is invalid
+ if (!IMD->hasAttrs())
+ return true;
+
const AttrVec &D = IMD->getAttrs();
if (D.size() != A.size())
return true;