LGTM with modifications
https://chromiumcodereview.appspot.com/10629004/diff/1/src/mark-compact.cc File src/mark-compact.cc (right): https://chromiumcodereview.appspot.com/10629004/diff/1/src/mark-compact.cc#newcode539 src/mark-compact.cc:539: // about choosing pages to free. We expect that halve empty pages halve empty -> half-empty https://chromiumcodereview.appspot.com/10629004/diff/1/src/mark-compact.cc#newcode548 src/mark-compact.cc:548: // try to free all mostly empty pages. We expect that empty pages that empty pages -> that almost empty pages ? https://chromiumcodereview.appspot.com/10629004/diff/1/src/mark-compact.cc#newcode552 src/mark-compact.cc:552: freeness_threshold = 80; I think we shouldn't set this so high. We might have 1/3 over-allocation and no pages are 80% free, but that shouldn't exclude them all from consideration. We pick the best 'n' below anyway. https://chromiumcodereview.appspot.com/10629004/diff/1/test/cctest/test-heap.cc File test/cctest/test-heap.cc (right): https://chromiumcodereview.appspot.com/10629004/diff/1/test/cctest/test-heap.cc#newcode1929 test/cctest/test-heap.cc:1929: // Triggering subsequent GCs should cause at least halve of the pages halve -> half https://chromiumcodereview.appspot.com/10629004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
