From: DeLesley Hutchins <delesley@google.com>
Date: Thu, 16 Feb 2012 17:15:51 +0000 (+0000)
Subject: Thread safety analysis: Don't check for lockable on undefined types.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=634b2930f5a8fc4b153437657ce786ca3fba5b1e;p=clang

Thread safety analysis: Don't check for lockable on undefined types.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@150702 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Sema/SemaDeclAttr.cpp b/lib/Sema/SemaDeclAttr.cpp
index 54e7969e0d..f0830b8861 100644
--- a/lib/Sema/SemaDeclAttr.cpp
+++ b/lib/Sema/SemaDeclAttr.cpp
@@ -283,6 +283,9 @@ static bool checkForLockableRecord(Sema &S, Decl *D, const AttributeList &Attr,
       << Attr.getName();
     return false;
   }
+  // Don't check for lockable if the class hasn't been defined yet. 
+  if (RT->isIncompleteType())
+    return true;
   // Flag error if the type is not lockable.
   if (!RT->getDecl()->getAttr<LockableAttr>()) {
     S.Diag(Attr.getLoc(), diag::err_attribute_argument_not_lockable)