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

Side by Side Diff: src/arm64/deoptimizer-arm64.cc

Issue 782703002: Cleanup deoptimizer relocation slot preparation. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Comment for x87 port. Created 6 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/arm/deoptimizer-arm.cc ('k') | src/compiler/arm/code-generator-arm.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 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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 #include "src/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/codegen.h" 7 #include "src/codegen.h"
8 #include "src/deoptimizer.h" 8 #include "src/deoptimizer.h"
9 #include "src/full-codegen.h" 9 #include "src/full-codegen.h"
10 #include "src/safepoint-table.h" 10 #include "src/safepoint-table.h"
11 11
12 12
13 namespace v8 { 13 namespace v8 {
14 namespace internal { 14 namespace internal {
15 15
16 16
17 int Deoptimizer::patch_size() { 17 int Deoptimizer::patch_size() {
18 // Size of the code used to patch lazy bailout points. 18 // Size of the code used to patch lazy bailout points.
19 // Patching is done by Deoptimizer::DeoptimizeFunction. 19 // Patching is done by Deoptimizer::DeoptimizeFunction.
20 return 4 * kInstructionSize; 20 return 4 * kInstructionSize;
21 } 21 }
22 22
23 23
24 void Deoptimizer::EnsureRelocSpaceForLazyDeoptimization(Handle<Code> code) {
25 // Empty because there is no need for relocation information for the code
26 // patching in Deoptimizer::PatchCodeForDeoptimization below.
27 }
28
24 29
25 void Deoptimizer::PatchCodeForDeoptimization(Isolate* isolate, Code* code) { 30 void Deoptimizer::PatchCodeForDeoptimization(Isolate* isolate, Code* code) {
26 // Invalidate the relocation information, as it will become invalid by the 31 // Invalidate the relocation information, as it will become invalid by the
27 // code patching below, and is not needed any more. 32 // code patching below, and is not needed any more.
28 code->InvalidateRelocation(); 33 code->InvalidateRelocation();
29 34
30 // TODO(jkummerow): if (FLAG_zap_code_space), make the code object's 35 // TODO(jkummerow): if (FLAG_zap_code_space), make the code object's
31 // entry sequence unusable (see other architectures). 36 // entry sequence unusable (see other architectures).
32 37
33 DeoptimizationInputData* deopt_data = 38 DeoptimizationInputData* deopt_data =
(...skipping 313 matching lines...) Expand 10 before | Expand all | Expand 10 after
347 352
348 void FrameDescription::SetCallerConstantPool(unsigned offset, intptr_t value) { 353 void FrameDescription::SetCallerConstantPool(unsigned offset, intptr_t value) {
349 // No out-of-line constant pool support. 354 // No out-of-line constant pool support.
350 UNREACHABLE(); 355 UNREACHABLE();
351 } 356 }
352 357
353 358
354 #undef __ 359 #undef __
355 360
356 } } // namespace v8::internal 361 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/arm/deoptimizer-arm.cc ('k') | src/compiler/arm/code-generator-arm.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698