diff options
author | Chris Lattner <sabre@nondot.org> | 2009-04-21 06:01:00 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2009-04-21 06:01:00 +0000 |
commit | 46c3c4ba78766ac0f1c5ec631b424773e21f5271 (patch) | |
tree | c3d48be8a96e6d96359d30a51bb6fec998700fa3 /lib/Sema/JumpDiagnostics.cpp | |
parent | 62f86c4555e14456acef6b251fcb13a66c3ce6dd (diff) |
add support for goto checking and @synchronized blocks,
rdar://6810106
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@69667 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/JumpDiagnostics.cpp')
-rw-r--r-- | lib/Sema/JumpDiagnostics.cpp | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/lib/Sema/JumpDiagnostics.cpp b/lib/Sema/JumpDiagnostics.cpp index 82db0bbd2c..b812f06f76 100644 --- a/lib/Sema/JumpDiagnostics.cpp +++ b/lib/Sema/JumpDiagnostics.cpp @@ -168,6 +168,22 @@ void JumpScopeChecker::BuildScopeInformation(Stmt *S, unsigned ParentScope) { continue; } + // Disallow jumps into the protected statement of an @synchronized, but + // allow jumps into the object expression it protects. + if (ObjCAtSynchronizedStmt *AS = dyn_cast<ObjCAtSynchronizedStmt>(SubStmt)){ + // Recursively walk the AST for the @synchronized object expr, it is + // evaluated in the normal scope. + BuildScopeInformation(AS->getSynchExpr(), ParentScope); + + // Recursively walk the AST for the @synchronized part, protected by a new + // scope. + Scopes.push_back(GotoScope(ParentScope, + diag::note_protected_by_objc_synchronized, + AS->getAtSynchronizedLoc())); + BuildScopeInformation(AS->getSynchBody(), Scopes.size()-1); + continue; + } + // Recursively walk the AST. BuildScopeInformation(SubStmt, ParentScope); } |