LGTM (with two nits).
https://chromiumcodereview.appspot.com/10919261/diff/1/src/ia32/lithium-ia32.h File src/ia32/lithium-ia32.h (right): https://chromiumcodereview.appspot.com/10919261/diff/1/src/ia32/lithium-ia32.h#newcode471 src/ia32/lithium-ia32.h:471: DECLARE_CONCRETE_INSTRUCTION(WrapReceiver, "wrap-receiver") Move this macro below the operand accessors. https://chromiumcodereview.appspot.com/10919261/diff/1/src/ia32/lithium-ia32.h#newcode524 src/ia32/lithium-ia32.h:524: DECLARE_CONCRETE_INSTRUCTION(ArgumentsLength, "arguments-length") Likewise. https://chromiumcodereview.appspot.com/10919261/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
