diff options
author | DeLesley Hutchins <delesley@google.com> | 2013-08-15 15:31:25 +0000 |
---|---|---|
committer | DeLesley Hutchins <delesley@google.com> | 2013-08-15 15:31:25 +0000 |
commit | 16b4ff5c4dba74988d8b72050f988a5e152a32fc (patch) | |
tree | 66330bc9636e035f1543ffd579bf0a1f8d409f79 /clang/lib/Analysis/ThreadSafety.cpp | |
parent | 24bef98f9685e8724fe3a2a72f7e013502a0fbb6 (diff) | |
download | bcm5719-llvm-16b4ff5c4dba74988d8b72050f988a5e152a32fc.tar.gz bcm5719-llvm-16b4ff5c4dba74988d8b72050f988a5e152a32fc.zip |
Thread safety analysis: move warnings within lock/unlock functions out of beta.
llvm-svn: 188465
Diffstat (limited to 'clang/lib/Analysis/ThreadSafety.cpp')
-rw-r--r-- | clang/lib/Analysis/ThreadSafety.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/clang/lib/Analysis/ThreadSafety.cpp b/clang/lib/Analysis/ThreadSafety.cpp index 7165aae4986..0c90751155d 100644 --- a/clang/lib/Analysis/ThreadSafety.cpp +++ b/clang/lib/Analysis/ThreadSafety.cpp @@ -2352,8 +2352,6 @@ void ThreadSafetyAnalyzer::runAnalysis(AnalysisDeclContext &AC) { = dyn_cast<SharedLocksRequiredAttr>(Attr)) { getMutexIDs(SharedLocksToAdd, A, (Expr*) 0, D); } else if (UnlockFunctionAttr *A = dyn_cast<UnlockFunctionAttr>(Attr)) { - if (!Handler.issueBetaWarnings()) - return; // UNLOCK_FUNCTION() is used to hide the underlying lock implementation. // We must ignore such methods. if (A->args_size() == 0) @@ -2363,15 +2361,11 @@ void ThreadSafetyAnalyzer::runAnalysis(AnalysisDeclContext &AC) { getMutexIDs(LocksReleased, A, (Expr*) 0, D); } else if (ExclusiveLockFunctionAttr *A = dyn_cast<ExclusiveLockFunctionAttr>(Attr)) { - if (!Handler.issueBetaWarnings()) - return; if (A->args_size() == 0) return; getMutexIDs(ExclusiveLocksAcquired, A, (Expr*) 0, D); } else if (SharedLockFunctionAttr *A = dyn_cast<SharedLockFunctionAttr>(Attr)) { - if (!Handler.issueBetaWarnings()) - return; if (A->args_size() == 0) return; getMutexIDs(SharedLocksAcquired, A, (Expr*) 0, D); |