http://codereview.chromium.org/2817018/diff/1/6 File src/compilation-cache.cc (right):
http://codereview.chromium.org/2817018/diff/1/6#newcode188 src/compilation-cache.cc:188: if (stats_table_ == NULL) { No, we can not use Isolate::Current()->stats_table() here: before change script_histogram is local static variable initialized once, to do the same with Isolates we initialize stats_table_ once and request script_histogram_ once. So stats_table_ is also a flag of initialization with some performance benefit. http://codereview.chromium.org/2817018/diff/1/6#newcode190 src/compilation-cache.cc:190: script_histogram_ = stats_table_->CreateHistogram( Because script_histogram has static scope with initialized once value, please, see explanation above http://codereview.chromium.org/2817018/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
