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

Side by Side Diff: src/x87/lithium-codegen-x87.cc

Issue 716463003: X87: fix one GC issue. (Closed) Base URL: https://chromium.googlesource.com/external/v8.git@bleeding_edge
Patch Set: Created 6 years, 1 month 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 | « no previous file | no next file » | 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 #include "src/v8.h" 5 #include "src/v8.h"
6 6
7 #if V8_TARGET_ARCH_X87 7 #if V8_TARGET_ARCH_X87
8 8
9 #include "src/base/bits.h" 9 #include "src/base/bits.h"
10 #include "src/code-factory.h" 10 #include "src/code-factory.h"
(...skipping 4046 matching lines...) Expand 10 before | Expand all | Expand 10 after
4057 __ fstp_d(FieldOperand(temp_result, HeapNumber::kValueOffset)); 4057 __ fstp_d(FieldOperand(temp_result, HeapNumber::kValueOffset));
4058 4058
4059 { 4059 {
4060 // Preserve the value of all registers. 4060 // Preserve the value of all registers.
4061 PushSafepointRegistersScope scope(this); 4061 PushSafepointRegistersScope scope(this);
4062 4062
4063 __ mov(esi, Operand(ebp, StandardFrameConstants::kContextOffset)); 4063 __ mov(esi, Operand(ebp, StandardFrameConstants::kContextOffset));
4064 __ push(temp_result); 4064 __ push(temp_result);
4065 __ CallRuntimeSaveDoubles(Runtime::kMathSqrtRT); 4065 __ CallRuntimeSaveDoubles(Runtime::kMathSqrtRT);
4066 RecordSafepointWithRegisters( 4066 RecordSafepointWithRegisters(
4067 instr->pointer_map(), 0, Safepoint::kNoLazyDeopt); 4067 instr->pointer_map(), 1, Safepoint::kNoLazyDeopt);
4068 __ StoreToSafepointRegisterSlot(temp_result, eax); 4068 __ StoreToSafepointRegisterSlot(temp_result, eax);
4069 } 4069 }
4070 X87PrepareToWrite(result_reg); 4070 X87PrepareToWrite(result_reg);
4071 // return value of MathExpRT is Smi or Heap Number. 4071 // return value of MathExpRT is Smi or Heap Number.
4072 __ JumpIfSmi(temp_result, &smi); 4072 __ JumpIfSmi(temp_result, &smi);
4073 // Heap number(double) 4073 // Heap number(double)
4074 __ fld_d(FieldOperand(temp_result, HeapNumber::kValueOffset)); 4074 __ fld_d(FieldOperand(temp_result, HeapNumber::kValueOffset));
4075 __ jmp(&finish); 4075 __ jmp(&finish);
4076 // SMI 4076 // SMI
4077 __ bind(&smi); 4077 __ bind(&smi);
(...skipping 193 matching lines...) Expand 10 before | Expand all | Expand 10 after
4271 X87LoadForUsage(input); 4271 X87LoadForUsage(input);
4272 __ fstp_d(FieldOperand(temp_result, HeapNumber::kValueOffset)); 4272 __ fstp_d(FieldOperand(temp_result, HeapNumber::kValueOffset));
4273 4273
4274 { 4274 {
4275 // Preserve the value of all registers. 4275 // Preserve the value of all registers.
4276 PushSafepointRegistersScope scope(this); 4276 PushSafepointRegistersScope scope(this);
4277 4277
4278 __ mov(esi, Operand(ebp, StandardFrameConstants::kContextOffset)); 4278 __ mov(esi, Operand(ebp, StandardFrameConstants::kContextOffset));
4279 __ push(temp_result); 4279 __ push(temp_result);
4280 __ CallRuntimeSaveDoubles(Runtime::kMathExpRT); 4280 __ CallRuntimeSaveDoubles(Runtime::kMathExpRT);
4281 RecordSafepointWithRegisters(instr->pointer_map(), 0, 4281 RecordSafepointWithRegisters(instr->pointer_map(), 1,
4282 Safepoint::kNoLazyDeopt); 4282 Safepoint::kNoLazyDeopt);
4283 __ StoreToSafepointRegisterSlot(temp_result, eax); 4283 __ StoreToSafepointRegisterSlot(temp_result, eax);
4284 } 4284 }
4285 X87PrepareToWrite(result_reg); 4285 X87PrepareToWrite(result_reg);
4286 // return value of MathExpRT is Smi or Heap Number. 4286 // return value of MathExpRT is Smi or Heap Number.
4287 __ JumpIfSmi(temp_result, &smi); 4287 __ JumpIfSmi(temp_result, &smi);
4288 // Heap number(double) 4288 // Heap number(double)
4289 __ fld_d(FieldOperand(temp_result, HeapNumber::kValueOffset)); 4289 __ fld_d(FieldOperand(temp_result, HeapNumber::kValueOffset));
4290 __ jmp(&finish); 4290 __ jmp(&finish);
4291 // SMI 4291 // SMI
(...skipping 2005 matching lines...) Expand 10 before | Expand all | Expand 10 after
6297 CallRuntime(Runtime::kPushBlockContext, 2, instr); 6297 CallRuntime(Runtime::kPushBlockContext, 2, instr);
6298 RecordSafepoint(Safepoint::kNoLazyDeopt); 6298 RecordSafepoint(Safepoint::kNoLazyDeopt);
6299 } 6299 }
6300 6300
6301 6301
6302 #undef __ 6302 #undef __
6303 6303
6304 } } // namespace v8::internal 6304 } } // namespace v8::internal
6305 6305
6306 #endif // V8_TARGET_ARCH_X87 6306 #endif // V8_TARGET_ARCH_X87
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698