Revision: 25263
Author:   [email protected]
Date:     Tue Nov 11 14:27:26 2014 UTC
Log:      Remove V8.ZoneSegmentBytes.

1) It's unused. 2) Since Zones can be used on a background thread, the Zone
cannot just access this counter from the Isolate, so keeping this counter up to
date would require extra work. Better to just remove it.

BUG=431860
LOG=n
[email protected]

Review URL: https://codereview.chromium.org/716793002
https://code.google.com/p/v8/source/detail?r=25263

Modified:
 /branches/bleeding_edge/src/counters.h
 /branches/bleeding_edge/src/zone-inl.h

=======================================
--- /branches/bleeding_edge/src/counters.h      Mon Sep 22 17:19:19 2014 UTC
+++ /branches/bleeding_edge/src/counters.h      Tue Nov 11 14:27:26 2014 UTC
@@ -489,7 +489,6 @@
SC(for_in, V8.ForIn) \ SC(enum_cache_hits, V8.EnumCacheHits) \ SC(enum_cache_misses, V8.EnumCacheMisses) \ - SC(zone_segment_bytes, V8.ZoneSegmentBytes) \ SC(fast_new_closure_total, V8.FastNewClosureTotal) \ SC(fast_new_closure_try_optimized, V8.FastNewClosureTryOptimized) \ SC(fast_new_closure_install_optimized, V8.FastNewClosureInstallOptimized) \
=======================================
--- /branches/bleeding_edge/src/zone-inl.h      Fri Aug  8 08:13:06 2014 UTC
+++ /branches/bleeding_edge/src/zone-inl.h      Tue Nov 11 14:27:26 2014 UTC
@@ -31,7 +31,6 @@

 void Zone::adjust_segment_bytes_allocated(int delta) {
   segment_bytes_allocated_ += delta;
- isolate_->counters()->zone_segment_bytes()->Set(segment_bytes_allocated_);
 }


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to