Landed.

http://codereview.chromium.org/10692043/diff/4001/src/hydrogen.cc
File src/hydrogen.cc (right):

http://codereview.chromium.org/10692043/diff/4001/src/hydrogen.cc#newcode5861
src/hydrogen.cc:5861: new(zone()) HCheckMaps(object, maps, zone());
On 2012/06/29 18:42:41, Jakob wrote:
nit: fits on one line

Done.

http://codereview.chromium.org/10692043/diff/4001/src/hydrogen.cc#newcode5865
src/hydrogen.cc:5865: check_maps, most_general_consolidated_map, false);
On 2012/06/29 18:42:41, Jakob wrote:
nit: fits on the same line as "object, key, val".

Done.

http://codereview.chromium.org/10692043/diff/4001/src/hydrogen.cc#newcode5949
src/hydrogen.cc:5949: : BuildLoadKeyedGeneric(object, key));
On 2012/06/29 18:42:41, Jakob wrote:
nit: why this change?

Done.

http://codereview.chromium.org/10692043/

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

Reply via email to