LGTM.
http://codereview.chromium.org/661468/diff/1/3 File src/objects.cc (right): http://codereview.chromium.org/661468/diff/1/3#newcode6949 src/objects.cc:6949: // 25% is still free after adding n elements and Update comment to reflect 50%. http://codereview.chromium.org/661468/diff/1/3#newcode6956 src/objects.cc:6956: const int kMinCapacityForPreTenure = 256; PreTenure -> Pretenure (to match the rest of the code). Maybe this should be kMaxCapacityForNonTenured? You can have pretenured hash tables with a smaller capacity than this. http://codereview.chromium.org/661468 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
