LGTM (just one small nit).
https://chromiumcodereview.appspot.com/10387231/diff/6001/src/objects-inl.h File src/objects-inl.h (right): https://chromiumcodereview.appspot.com/10387231/diff/6001/src/objects-inl.h#newcode1948 src/objects-inl.h:1948: ASSERT(!HEAP->InNewSpace(heap->null_value())); That assertion is already in FixedArray::set_null_unchecked, no need to duplicate it here. https://chromiumcodereview.appspot.com/10387231/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
