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

Unified Diff: runtime/vm/assembler_x64.cc

Issue 10919008: Unbox phis that were proven to be of type Double. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: address Srdjan's comments Created 8 years, 4 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/assembler_ia32.cc ('k') | runtime/vm/code_generator.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/assembler_x64.cc
diff --git a/runtime/vm/assembler_x64.cc b/runtime/vm/assembler_x64.cc
index bc213eb74b1a16b3f3945b80d9bcb3a2e6847705..600b6c9bd8be4703f2621d6a189153aedb8ee61a 100644
--- a/runtime/vm/assembler_x64.cc
+++ b/runtime/vm/assembler_x64.cc
@@ -1693,15 +1693,35 @@ static const Register volatile_cpu_registers[kNumberOfVolatileCpuRegisters] = {
RAX, RCX, RDX, RSI, RDI, R8, R9, R10, R11
};
+// XMM0 is used only as a scratch register in the optimized code. No need to
+// save it.
+static const intptr_t kNumberOfVolatileXmmRegisters =
+ kNumberOfXmmRegisters - 1;
+
+
+static const intptr_t kNumberOfVolatileRegisters =
+ kNumberOfVolatileCpuRegisters + kNumberOfVolatileXmmRegisters;
+
void Assembler::EnterCallRuntimeFrame(intptr_t frame_space) {
enter(Immediate(0));
- // Preserve volatile registers.
+ // Preserve volatile CPU registers.
for (intptr_t i = 0; i < kNumberOfVolatileCpuRegisters; i++) {
pushq(volatile_cpu_registers[i]);
}
+ // Preserve all XMM registers except XMM0
+ subq(RSP, Immediate((kNumberOfXmmRegisters - 1) * kDoubleSize));
+ // Store XMM registers with the lowest register number at the lowest
+ // address.
+ intptr_t offset = 0;
+ for (intptr_t reg_idx = 1; reg_idx < kNumberOfXmmRegisters; ++reg_idx) {
+ XmmRegister xmm_reg = static_cast<XmmRegister>(reg_idx);
+ movsd(Address(RSP, offset), xmm_reg);
+ offset += kDoubleSize;
+ }
+
ReserveAlignedFrameSpace(frame_space);
}
@@ -1710,9 +1730,19 @@ void Assembler::LeaveCallRuntimeFrame() {
// RSP might have been modified to reserve space for arguments
// and ensure proper alignment of the stack frame.
// We need to restore it before restoring registers.
- leaq(RSP, Address(RBP, -kNumberOfVolatileCpuRegisters * kWordSize));
+ leaq(RSP, Address(RBP, -kNumberOfVolatileRegisters * kWordSize));
+
+ // Restore all XMM registers except XMM0
+ // XMM registers have the lowest register number at the lowest address.
+ intptr_t offset = 0;
+ for (intptr_t reg_idx = 1; reg_idx < kNumberOfXmmRegisters; ++reg_idx) {
+ XmmRegister xmm_reg = static_cast<XmmRegister>(reg_idx);
+ movsd(xmm_reg, Address(RSP, offset));
+ offset += kDoubleSize;
+ }
+ addq(RSP, Immediate(offset));
- // Restore volatile registers.
+ // Restore volatile CPU registers.
for (intptr_t i = kNumberOfVolatileCpuRegisters - 1; i >= 0; i--) {
popq(volatile_cpu_registers[i]);
}
« no previous file with comments | « runtime/vm/assembler_ia32.cc ('k') | runtime/vm/code_generator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698