LGTM
http://codereview.chromium.org/6119009/diff/6001/include/v8.h File include/v8.h (right): http://codereview.chromium.org/6119009/diff/6001/include/v8.h#newcode3356 include/v8.h:3356: template <size_t ptr_size> struct SmiTraits; I think I would prefer "SmiTagging", but it's better than SmiConstants. http://codereview.chromium.org/6119009/diff/6001/include/v8.h#newcode3386 include/v8.h:3386: // alignment. Currently we require 8 bytes alignment (which Drop "better" alignment (the text should be stand-alone, so it should not be implicitly relative to the 32-bit case). Just say something like "To maximize the range of pointers that can be encoded in the available 32 bit, we require them to be 8-byte aligned". http://codereview.chromium.org/6119009/diff/6001/include/v8.h#newcode3393 include/v8.h:3393: intptr_t(0xffffffff) << kPointerAlignment; Shouldn't this be inverted to mask the bits that must be zero? http://codereview.chromium.org/6119009/diff/6001/test/cctest/test-api.cc File test/cctest/test-api.cc (right): http://codereview.chromium.org/6119009/diff/6001/test/cctest/test-api.cc#newcode829 test/cctest/test-api.cc:829: *value = 42; Try storing both values that can and can't be stored as smis. http://codereview.chromium.org/6119009/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
