LGTM - it's kinda difficult to review with all this moving code around, but it looks good.
http://codereview.chromium.org/6527/diff/203/6 File src/codegen-ia32.cc (right): http://codereview.chromium.org/6527/diff/203/6#newcode216 Line 216: Operand SlotOperand(Slot* slot, Register tmp); I guess there's a good reason why this isn't const and the other operand helpers are? http://codereview.chromium.org/6527 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
