LGTM

http://codereview.chromium.org/527006/diff/1/2
File src/heap.cc (right):

http://codereview.chromium.org/527006/diff/1/2#newcode1611
src/heap.cc:1611: // max_semispace_size_ == 512 KB =>
number_string_cache_size = 16.
32?

Also, the logic seems different from what the comment suggests.  Update
the comment?

http://codereview.chromium.org/527006
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to