LGTM.
https://chromiumcodereview.appspot.com/9594013/diff/1/src/heap.cc File src/heap.cc (right): https://chromiumcodereview.appspot.com/9594013/diff/1/src/heap.cc#newcode1933 src/heap.cc:1933: accessors->set_setter(the_hole_value(), SKIP_WRITE_BARRIER); It would be possible to add an AllocateStructWithFiller() that takes another initial filler object (besides undefined). This would avoid two unnecessary writes. But I suppose that's not worth the effort here. So I am fine with leaving everything as it is. https://chromiumcodereview.appspot.com/9594013/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
