Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: runtime/vm/isolate.cc

Issue 2118813003: Separate mutex for megamorphic cache tables lookup in the background compiler. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« runtime/vm/isolate.h ('K') | « runtime/vm/isolate.h ('k') | runtime/vm/megamorphic_cache_table.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/isolate.cc
diff --git a/runtime/vm/isolate.cc b/runtime/vm/isolate.cc
index bd26039f1e8b524f271d2dc1c6a53b41cff88847..68d22ed3efa2a7cb0640a6c5205fd40db8e4b7a9 100644
--- a/runtime/vm/isolate.cc
+++ b/runtime/vm/isolate.cc
@@ -797,6 +797,7 @@ Isolate::Isolate(const Dart_IsolateFlags& api_flags)
symbols_mutex_(new Mutex()),
type_canonicalization_mutex_(new Mutex()),
constant_canonicalization_mutex_(new Mutex()),
+ lookup_mutex_(new Mutex()),
Cutch 2016/07/01 17:16:18 where is this mutex deleted?
siva 2016/07/01 17:39:29 Yes we need to delete it in the destructor.
Florian Schneider 2016/07/01 19:31:58 Done.
Florian Schneider 2016/07/01 19:31:58 Done.
message_handler_(NULL),
spawn_state_(NULL),
is_runnable_(false),
« runtime/vm/isolate.h ('K') | « runtime/vm/isolate.h ('k') | runtime/vm/megamorphic_cache_table.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698