LGTM (with a nit).
http://codereview.chromium.org/9086005/diff/1/src/spaces.cc File src/spaces.cc (right): http://codereview.chromium.org/9086005/diff/1/src/spaces.cc#newcode1272 src/spaces.cc:1272: if (!is_committed()) { Can we invert the condition and place the block below inside the if statement? I like that better than early returns. http://codereview.chromium.org/9086005/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
