diff options
Diffstat (limited to 'deps/v8/src/codegen/ia32/assembler-ia32.cc')
-rw-r--r-- | deps/v8/src/codegen/ia32/assembler-ia32.cc | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/deps/v8/src/codegen/ia32/assembler-ia32.cc b/deps/v8/src/codegen/ia32/assembler-ia32.cc index 321a59cede..f19c8dd1cd 100644 --- a/deps/v8/src/codegen/ia32/assembler-ia32.cc +++ b/deps/v8/src/codegen/ia32/assembler-ia32.cc @@ -302,6 +302,15 @@ Assembler::Assembler(const AssemblerOptions& options, void Assembler::GetCode(Isolate* isolate, CodeDesc* desc, SafepointTableBuilder* safepoint_table_builder, int handler_table_offset) { + // As a crutch to avoid having to add manual Align calls wherever we use a + // raw workflow to create Code objects (mostly in tests), add another Align + // call here. It does no harm - the end of the Code object is aligned to the + // (larger) kCodeAlignment anyways. + // TODO(jgruber): Consider moving responsibility for proper alignment to + // metadata table builders (safepoint, handler, constant pool, code + // comments). + DataAlign(Code::kMetadataAlignment); + const int code_comments_size = WriteCodeComments(); // Finalize code (at this point overflow() may be true, but the gap ensures @@ -510,13 +519,6 @@ void Assembler::pop(Operand dst) { emit_operand(eax, dst); } -void Assembler::enter(const Immediate& size) { - EnsureSpace ensure_space(this); - EMIT(0xC8); - emit_w(size); - EMIT(0); -} - void Assembler::leave() { EnsureSpace ensure_space(this); EMIT(0xC9); |