Revision: 14338
Author:   [email protected]
Date:     Thu Apr 18 08:09:48 2013
Log: Saving Hydrogen code stubs in the snapshot requires us to respect the availability of VFP32DREGS.

BUG=

Review URL: https://codereview.chromium.org/13878011
http://code.google.com/p/v8/source/detail?r=14338

Modified:
 /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc

=======================================
--- /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Thu Apr 18 06:45:19 2013 +++ /branches/bleeding_edge/src/arm/lithium-codegen-arm.cc Thu Apr 18 08:09:48 2013
@@ -36,6 +36,14 @@
 namespace internal {


+static SaveFPRegsMode GetSaveFPRegsMode() {
+  // We don't need to save floating point regs when generating the snapshot
+  return CpuFeatures::IsSafeForSnapshot(VFP32DREGS)
+      ? kSaveFPRegs
+      : kDontSaveFPRegs;
+}
+
+
 class SafepointGenerator : public CallWrapper {
  public:
   SafepointGenerator(LCodeGen* codegen,
@@ -238,7 +246,12 @@
         __ str(r0, target);
         // Update the write barrier. This clobbers r3 and r0.
         __ RecordWriteContextSlot(
- cp, target.offset(), r0, r3, GetLinkRegisterState(), kSaveFPRegs);
+            cp,
+            target.offset(),
+            r0,
+            r3,
+            GetLinkRegisterState(),
+            GetSaveFPRegsMode());
       }
     }
     Comment(";;; End allocate local context");
@@ -3067,7 +3080,7 @@
                               value,
                               scratch,
                               GetLinkRegisterState(),
-                              kSaveFPRegs,
+                              GetSaveFPRegsMode(),
                               EMIT_REMEMBERED_SET,
                               check_needed);
   }
@@ -4256,7 +4269,7 @@
                           scratch,
                           temp,
                           GetLinkRegisterState(),
-                          kSaveFPRegs,
+                          GetSaveFPRegsMode(),
                           OMIT_REMEMBERED_SET,
                           OMIT_SMI_CHECK);
     }
@@ -4275,7 +4288,7 @@
                           value,
                           scratch,
                           GetLinkRegisterState(),
-                          kSaveFPRegs,
+                          GetSaveFPRegsMode(),
                           EMIT_REMEMBERED_SET,
                           check_needed);
     }
@@ -4290,7 +4303,7 @@
                           value,
                           object,
                           GetLinkRegisterState(),
-                          kSaveFPRegs,
+                          GetSaveFPRegsMode(),
                           EMIT_REMEMBERED_SET,
                           check_needed);
     }
@@ -4484,7 +4497,7 @@
                    key,
                    value,
                    GetLinkRegisterState(),
-                   kSaveFPRegs,
+                   GetSaveFPRegsMode(),
                    EMIT_REMEMBERED_SET,
                    check_needed);
   }

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to