https://codereview.chromium.org/66843011/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/66843011/diff/1/src/hydrogen.cc#newcode7953
src/hydrogen.cc:7953: byte_offset);
On 2013/11/18 08:02:31, Benedikt Meurer wrote:
byte_offset should be a SMI, so you should pass Representation::Smi()
to
ForJSObjectOffset() to avoid the write barrier.

Byte offset is not neccessarily a smi here. Ditto byte length below.

https://codereview.chromium.org/66843011/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to