LGTM
http://codereview.chromium.org/600123/diff/1/3 File src/x64/ic-x64.cc (right): http://codereview.chromium.org/600123/diff/1/3#newcode588 src/x64/ic-x64.cc:588: // Check that the receiver isn't a smi. Not sure that the order of these checks matter much, but they are reversed here compared to the ARM version. Would be nice to be consistent. It is probably most likely that the index is not a smi, so maybe check that first on all platforms? http://codereview.chromium.org/600123 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
