Thanks. Uploaded new PS.
https://codereview.chromium.org/1218863002/diff/180001/src/heap/cleanup-gc.cc
File src/heap/cleanup-gc.cc (right):
https://codereview.chromium.org/1218863002/diff/180001/src/heap/cleanup-gc.cc#newcode132
src/heap/cleanup-gc.cc:132: void CleanupGC::ScheduleTimer(double
delay_ms) {
On 2015/07/01 12:03:16, Hannes Payer wrote:
Can we DCHECK that delay_ms is always positive.
Done.
https://codereview.chromium.org/1218863002/diff/180001/src/heap/cleanup-gc.cc#newcode137
src/heap/cleanup-gc.cc:137: isolate, new CleanupGC::TimerTask(this),
(delay_ms + kSlackMs) / 1000.0);
On 2015/07/01 12:03:16, Hannes Payer wrote:
where is the delete?
Platform will delete it.
https://codereview.chromium.org/1218863002/diff/180001/src/heap/cleanup-gc.h
File src/heap/cleanup-gc.h (right):
https://codereview.chromium.org/1218863002/diff/180001/src/heap/cleanup-gc.h#newcode17
src/heap/cleanup-gc.h:17: // The goal of the CleanupGC class is to
detect transition of the mutator
On 2015/07/01 12:03:16, Hannes Payer wrote:
Hmm... I am not a big fan of the name CleanupGC. What about
MemoryController? I
am open to other alternatives...
Renamed to MemoryReducer as discussed.
https://codereview.chromium.org/1218863002/diff/180001/src/heap/gc-idle-time-handler.cc
File src/heap/gc-idle-time-handler.cc (right):
https://codereview.chromium.org/1218863002/diff/180001/src/heap/gc-idle-time-handler.cc#newcode211
src/heap/gc-idle-time-handler.cc:211: // (2) If the context disposal
rate is high and we cannot perform a full GC,
On 2015/07/01 12:03:17, Hannes Payer wrote:
please update this comment
Done.
https://codereview.chromium.org/1218863002/diff/180001/src/heap/heap.cc
File src/heap/heap.cc (right):
https://codereview.chromium.org/1218863002/diff/180001/src/heap/heap.cc#newcode4883
src/heap/heap.cc:4883: CollectAllGarbage(kReduceMemoryFootprintMask,
On 2015/07/01 12:03:17, Hannes Payer wrote:
Could this call end an ongoing incremental marking cycle?
I removed this completely and added DCHECK(context_disposed_ > 0).
https://codereview.chromium.org/1218863002/
--
--
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.
