On 2012/05/02 11:38:34, Yang wrote:
Integers are not always converted to Smis by a left-shift by one. While we can assume that on platform-dependent code (ia32, arm and mips), we avoid using
that
assumption on platform-independent code because on x64 a Smi is converted by a left-shift by 32. Code that intend to store integers are Smis should therefore
use Smi::FromInt() and Smi::cast()->value().

Thanks a lot for the very accurate review. I hope I addressed all your comments.
Plus add added a test (currently disabled because if issue 915).

http://codereview.chromium.org/10263002/

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

Reply via email to