LGTM!
http://codereview.chromium.org/7621014/diff/5001/src/heap.cc File src/heap.cc (right): http://codereview.chromium.org/7621014/diff/5001/src/heap.cc#newcode738 src/heap.cc:738: old_gen_limit_factor_ = 1; I think you wanted 2 here. http://codereview.chromium.org/7621014/diff/5001/src/heap.h File src/heap.h (right): http://codereview.chromium.org/7621014/diff/5001/src/heap.h#newcode1233 src/heap.h:1233: limit += new_space_.Capacity(); new_space_.Capacity can change while lazy sweeping is in progress. So Lower... will fluctuate. Is it intentional? http://codereview.chromium.org/7621014/diff/5001/src/spaces.cc File src/spaces.cc (right): http://codereview.chromium.org/7621014/diff/5001/src/spaces.cc#newcode1933 src/spaces.cc:1933: if (FLAG_trace_gc && first_unswept_page_ != NULL) { _verbose http://codereview.chromium.org/7621014/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
