Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(15)

Side by Side Diff: src/x87/macro-assembler-x87.cc

Issue 1476763002: Make whether or not a Code object should be created by masm explicit (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: enum class FTW Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/x87/macro-assembler-x87.h ('k') | test/cctest/test-assembler-arm64.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #if V8_TARGET_ARCH_X87 5 #if V8_TARGET_ARCH_X87
6 6
7 #include "src/base/bits.h" 7 #include "src/base/bits.h"
8 #include "src/base/division-by-constant.h" 8 #include "src/base/division-by-constant.h"
9 #include "src/bootstrapper.h" 9 #include "src/bootstrapper.h"
10 #include "src/codegen.h" 10 #include "src/codegen.h"
11 #include "src/debug/debug.h" 11 #include "src/debug/debug.h"
12 #include "src/runtime/runtime.h" 12 #include "src/runtime/runtime.h"
13 #include "src/x87/frames-x87.h" 13 #include "src/x87/frames-x87.h"
14 #include "src/x87/macro-assembler-x87.h" 14 #include "src/x87/macro-assembler-x87.h"
15 15
16 namespace v8 { 16 namespace v8 {
17 namespace internal { 17 namespace internal {
18 18
19 // ------------------------------------------------------------------------- 19 // -------------------------------------------------------------------------
20 // MacroAssembler implementation. 20 // MacroAssembler implementation.
21 21
22 MacroAssembler::MacroAssembler(Isolate* arg_isolate, void* buffer, int size) 22 MacroAssembler::MacroAssembler(Isolate* arg_isolate, void* buffer, int size,
23 CodeObjectRequired create_code_object)
23 : Assembler(arg_isolate, buffer, size), 24 : Assembler(arg_isolate, buffer, size),
24 generating_stub_(false), 25 generating_stub_(false),
25 has_frame_(false) { 26 has_frame_(false) {
26 if (isolate() != NULL) { 27 if (create_code_object == CodeObjectRequired::kYes) {
27 // TODO(titzer): should we just use a null handle here instead?
28 code_object_ = 28 code_object_ =
29 Handle<Object>::New(isolate()->heap()->undefined_value(), isolate()); 29 Handle<Object>::New(isolate()->heap()->undefined_value(), isolate());
30 } 30 }
31 } 31 }
32 32
33 33
34 void MacroAssembler::Load(Register dst, const Operand& src, Representation r) { 34 void MacroAssembler::Load(Register dst, const Operand& src, Representation r) {
35 DCHECK(!r.IsDouble()); 35 DCHECK(!r.IsDouble());
36 if (r.IsInteger8()) { 36 if (r.IsInteger8()) {
37 movsx_b(dst, src); 37 movsx_b(dst, src);
(...skipping 2618 matching lines...) Expand 10 before | Expand all | Expand 10 after
2656 int n_of_non_aliasing_regs = NumRegs(regs); 2656 int n_of_non_aliasing_regs = NumRegs(regs);
2657 2657
2658 return n_of_valid_regs != n_of_non_aliasing_regs; 2658 return n_of_valid_regs != n_of_non_aliasing_regs;
2659 } 2659 }
2660 #endif 2660 #endif
2661 2661
2662 2662
2663 CodePatcher::CodePatcher(byte* address, int size) 2663 CodePatcher::CodePatcher(byte* address, int size)
2664 : address_(address), 2664 : address_(address),
2665 size_(size), 2665 size_(size),
2666 masm_(NULL, address, size + Assembler::kGap) { 2666 masm_(NULL, address, size + Assembler::kGap, CodeObjectRequired::kNo) {
2667 // Create a new macro assembler pointing to the address of the code to patch. 2667 // Create a new macro assembler pointing to the address of the code to patch.
2668 // The size is adjusted with kGap on order for the assembler to generate size 2668 // The size is adjusted with kGap on order for the assembler to generate size
2669 // bytes of instructions without failing with buffer size constraints. 2669 // bytes of instructions without failing with buffer size constraints.
2670 DCHECK(masm_.reloc_info_writer.pos() == address_ + size_ + Assembler::kGap); 2670 DCHECK(masm_.reloc_info_writer.pos() == address_ + size_ + Assembler::kGap);
2671 } 2671 }
2672 2672
2673 2673
2674 CodePatcher::~CodePatcher() { 2674 CodePatcher::~CodePatcher() {
2675 // Indicate that code has changed. 2675 // Indicate that code has changed.
2676 Assembler::FlushICacheWithoutIsolate(address_, size_); 2676 Assembler::FlushICacheWithoutIsolate(address_, size_);
(...skipping 340 matching lines...) Expand 10 before | Expand all | Expand 10 after
3017 mov(eax, dividend); 3017 mov(eax, dividend);
3018 shr(eax, 31); 3018 shr(eax, 31);
3019 add(edx, eax); 3019 add(edx, eax);
3020 } 3020 }
3021 3021
3022 3022
3023 } // namespace internal 3023 } // namespace internal
3024 } // namespace v8 3024 } // namespace v8
3025 3025
3026 #endif // V8_TARGET_ARCH_X87 3026 #endif // V8_TARGET_ARCH_X87
OLDNEW
« no previous file with comments | « src/x87/macro-assembler-x87.h ('k') | test/cctest/test-assembler-arm64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698