diff options
| author | Stephen Hines <srhines@google.com> | 2018-02-06 18:35:41 +0000 |
|---|---|---|
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2018-02-06 18:35:41 +0000 |
| commit | 8bcecf915545581bf23b5348cd94d3f5ec6eb669 (patch) | |
| tree | 99146665aa76b27adb7d253cc8e14a48894207d4 /adbconnection | |
| parent | 045112c32ca814be43c5965de3b3006d313cfc90 (diff) | |
| parent | e11d50f23dc1256ad933d209f2db8f58138f02c0 (diff) | |
Merge "Workaround bogus Thread Safety Analysis warning"
Diffstat (limited to 'adbconnection')
| -rw-r--r-- | adbconnection/adbconnection.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/adbconnection/adbconnection.cc b/adbconnection/adbconnection.cc index 6b82be6193..a0c99663b4 100644 --- a/adbconnection/adbconnection.cc +++ b/adbconnection/adbconnection.cc @@ -555,7 +555,10 @@ bool AdbConnectionState::SetupAdbConnection() { void AdbConnectionState::RunPollLoop(art::Thread* self) { CHECK_NE(agent_name_, ""); CHECK_EQ(self->GetState(), art::kNative); - art::Locks::mutator_lock_->AssertNotHeld(self); + // TODO: Clang prebuilt for r316199 produces bogus thread safety analysis warning for holding both + // exclusive and shared lock in the same scope. Remove the assertion as a temporary workaround. + // http://b/71769596 + // art::Locks::mutator_lock_->AssertNotHeld(self); self->SetState(art::kWaitingInMainDebuggerLoop); // shutting_down_ set by StopDebuggerThreads while (!shutting_down_) { |