LGTM (with a nit).

https://chromiumcodereview.appspot.com/10802048/diff/1/src/objects.cc
File src/objects.cc (right):

https://chromiumcodereview.appspot.com/10802048/diff/1/src/objects.cc#newcode7159
src/objects.cc:7159: t->SetTarget(transition_index, t->GetTarget(i));
Can we have a one line comment above this line saying that we don't need
to record slots because we don't compact maps.

https://chromiumcodereview.appspot.com/10802048/

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

Reply via email to