LGTM.
http://codereview.chromium.org/155679/diff/1/6 File src/ia32/register-allocator-ia32-inl.h (right): http://codereview.chromium.org/155679/diff/1/6#newcode68 Line 68: const Register registers[] = { eax, ebx, ecx, edx, edi }; This static also occurs above in RegisterAllocator::ToNumber, and analogously in the ARM and x64 files. I wonder if we should change those too? http://codereview.chromium.org/155679 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
