LGTM
http://codereview.chromium.org/9452002/diff/1/src/mark-compact.cc File src/mark-compact.cc (right): http://codereview.chromium.org/9452002/diff/1/src/mark-compact.cc#newcode3077 src/mark-compact.cc:3077: int cell_index = I think it is clearer to move this down to immediately before the for loop. Especially in this case, but also in the other cases in this file. http://codereview.chromium.org/9452002/diff/1/src/platform-cygwin.cc File src/platform-cygwin.cc (right): http://codereview.chromium.org/9452002/diff/1/src/platform-cygwin.cc#newcode362 src/platform-cygwin.cc:362: PAGE_READONLY | PAGE_GUARD)) { Can we not map it with no permissions at all, instead of read only? http://codereview.chromium.org/9452002/diff/1/src/platform-freebsd.cc File src/platform-freebsd.cc (right): http://codereview.chromium.org/9452002/diff/1/src/platform-freebsd.cc#newcode416 src/platform-freebsd.cc:416: return true; You could have put this in the posix file. http://codereview.chromium.org/9452002/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
