http://codereview.chromium.org/469006/diff/1/3 File src/ia32/fast-codegen-ia32.cc (right):
http://codereview.chromium.org/469006/diff/1/3#newcode416 src/ia32/fast-codegen-ia32.cc:416: I'm confused by the variable rewrites again :) And I guess the following defensive assert holds here: ASSERT(slot != NULL || prop != NULL) http://codereview.chromium.org/469006/diff/1/3#newcode495 src/ia32/fast-codegen-ia32.cc:495: Handle<Code> ic(Builtins::builtin(Builtins::KeyedLoadIC_Initialize)); KeyedStore instead of load? http://codereview.chromium.org/469006/diff/1/2 File test/mjsunit/regress/regress-540.js (right): http://codereview.chromium.org/469006/diff/1/2#newcode36 test/mjsunit/regress/regress-540.js:36: (function (x) { This function won't hit the top-level compiler. You could use the flag --always_fast_compiler to do that. http://codereview.chromium.org/469006 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
