LGTM with a nit.

You'll need me to land this for you, right?


https://codereview.chromium.org/403003002/diff/1/src/deoptimizer.cc
File src/deoptimizer.cc (right):

https://codereview.chromium.org/403003002/diff/1/src/deoptimizer.cc#newcode3263
src/deoptimizer.cc:3263: #if V8_TARGET_LITTLE_ENDIAN ||
!V8_HOST_ARCH_64_BIT
nit: I think "V8_HOST_ARCH_32_BIT" would be slightly more appropriate
than "!V8_HOST_ARCH_64_BIT".

https://codereview.chromium.org/403003002/

--
--
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/d/optout.

Reply via email to