lgtm
http://codereview.chromium.org/8492004/diff/15004/src/ia32/deoptimizer-ia32.cc File src/ia32/deoptimizer-ia32.cc (right): http://codereview.chromium.org/8492004/diff/15004/src/ia32/deoptimizer-ia32.cc#newcode62 src/ia32/deoptimizer-ia32.cc:62: Address curr_reloc_address = code_start_address + pc_offset; You can just use prev_pc_offset instead calculating real address. pc_delta = pc_offset - prev_pc_offset; start address is irrelevant. http://codereview.chromium.org/8492004/diff/15004/src/ia32/deoptimizer-ia32.cc#newcode156 src/ia32/deoptimizer-ia32.cc:156: RelocInfo rinfo(curr_address + 1, // 1 after the call opcode. relocation info is not used? I think this is not GC safe http://codereview.chromium.org/8492004/diff/15004/src/ia32/lithium-codegen-ia32.cc File src/ia32/lithium-codegen-ia32.cc (right): http://codereview.chromium.org/8492004/diff/15004/src/ia32/lithium-codegen-ia32.cc#newcode1978 src/ia32/lithium-codegen-ia32.cc:1978: RecordPosition(instr->pointer_map()->position()); I think this can be rolled back into CallCodeGeneric because it seems to be just code duplication now. http://codereview.chromium.org/8492004/diff/15004/src/ia32/lithium-codegen-ia32.cc#newcode4419 src/ia32/lithium-codegen-ia32.cc:4419: void LCodeGen::DoLazyBailout(LLazyBailout* instr) { maybe instruction should be renamed to LazyDeopt? http://codereview.chromium.org/8492004/diff/15004/src/ia32/lithium-codegen-ia32.h File src/ia32/lithium-codegen-ia32.h (right): http://codereview.chromium.org/8492004/diff/15004/src/ia32/lithium-codegen-ia32.h#newcode215 src/ia32/lithium-codegen-ia32.h:215: void RegisterEnvironmentForDeoptimization(LEnvironment* environment, maybe use Safepoint::DeoptMode instead of boolean? http://codereview.chromium.org/8492004/diff/15004/src/lithium.h File src/lithium.h (right): http://codereview.chromium.org/8492004/diff/15004/src/lithium.h#newcode488 src/lithium.h:488: void Register(int deoptimization_index, int translation_index, one argument on a line http://codereview.chromium.org/8492004/diff/15004/src/safepoint-table.h File src/safepoint-table.h (right): http://codereview.chromium.org/8492004/diff/15004/src/safepoint-table.h#newcode71 src/safepoint-table.h:71: kDeoptIndexBits> {}; // NOLINT LONG LINE http://codereview.chromium.org/8492004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
