LGTM.

http://codereview.chromium.org/174052/diff/1/4
File src/spaces.cc (right):

http://codereview.chromium.org/174052/diff/1/4#newcode1083
Line 1083: int extra = RoundUp(capacity_ / 2, OS::AllocateAlignment());
How about using Min(RoundUp(...), maximum_capacity - capacity)?

http://codereview.chromium.org/174052

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to