diff options
author | DeLesley Hutchins <delesley@google.com> | 2012-05-02 17:38:37 +0000 |
---|---|---|
committer | DeLesley Hutchins <delesley@google.com> | 2012-05-02 17:38:37 +0000 |
commit | d96b46ace2aeca93002365ab572ded3742cf78fb (patch) | |
tree | 834bb15d00061bdbb12d1311e8ef5f37f95964c2 /clang/lib/Sema | |
parent | d7c6777d504013bee96bfe9a50f9e34b309218c8 (diff) | |
download | bcm5719-llvm-d96b46ace2aeca93002365ab572ded3742cf78fb.tar.gz bcm5719-llvm-d96b46ace2aeca93002365ab572ded3742cf78fb.zip |
Thread Safety Analysis: fixed attribute handling for lock_returned attribute.
llvm-svn: 156005
Diffstat (limited to 'clang/lib/Sema')
-rw-r--r-- | clang/lib/Sema/SemaDeclAttr.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaDeclAttr.cpp b/clang/lib/Sema/SemaDeclAttr.cpp index c8af3a430a2..13bf0aa4875 100644 --- a/clang/lib/Sema/SemaDeclAttr.cpp +++ b/clang/lib/Sema/SemaDeclAttr.cpp @@ -681,9 +681,14 @@ static void handleLockReturnedAttr(Sema &S, Decl *D, return; // check that the argument is lockable object - checkForLockableRecord(S, D, Attr, Arg->getType()); + SmallVector<Expr*, 1> Args; + checkAttrArgsAreLockableObjs(S, D, Attr, Args); + unsigned Size = Args.size(); + if (Size == 0) + return; - D->addAttr(::new (S.Context) LockReturnedAttr(Attr.getRange(), S.Context, Arg)); + D->addAttr(::new (S.Context) LockReturnedAttr(Attr.getRange(), S.Context, + Args[0])); } static void handleLocksExcludedAttr(Sema &S, Decl *D, |