Please review again. I have marked the places that are different from last time.
https://chromiumcodereview.appspot.com/9233050/diff/1004/src/heap.cc File src/heap.cc (right): https://chromiumcodereview.appspot.com/9233050/diff/1004/src/heap.cc#newcode591 src/heap.cc:591: while (gc_performed && counter++ < kThreshold) { This function was changed in response to the last set of comments. https://chromiumcodereview.appspot.com/9233050/diff/1004/src/spaces.cc File src/spaces.cc (right): https://chromiumcodereview.appspot.com/9233050/diff/1004/src/spaces.cc#newcode286 src/spaces.cc:286: CHECK_EQ(memory_allocator_reserved_, static_cast<intptr_t>(0)); Cast added here and a few other places to make the Windows compiler happy. https://chromiumcodereview.appspot.com/9233050/diff/1004/src/spaces.cc#newcode848 src/spaces.cc:848: // We initially only commit a part of the page. The deserialization of the I updated the comment. https://chromiumcodereview.appspot.com/9233050/diff/1004/src/spaces.cc#newcode1928 src/spaces.cc:1928: *list = node; This assignment was added. https://chromiumcodereview.appspot.com/9233050/diff/1004/src/store-buffer.cc File src/store-buffer.cc (right): https://chromiumcodereview.appspot.com/9233050/diff/1004/src/store-buffer.cc#newcode564 src/store-buffer.cc:564: if ((start_integer & (Page::kGrowthUnit - 1)) == 0 && This 'if' and its contents are new. https://chromiumcodereview.appspot.com/9233050/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
