[v8-dev] ARM: safepoints frame optimization

src/arm/assembler-arm.h

Index: src/arm/assembler-arm.h
diff --git a/src/arm/assembler-arm.h b/src/arm/assembler-arm.h
index ccb5104206883e9891fce042208538cd8dab2613..31796dcb5c214bd50eac4a6597eb34ebf287308e 100644
--- a/src/arm/assembler-arm.h
+++ b/src/arm/assembler-arm.h
@@ -1446,6 +1446,14 @@ class Assembler : public AssemblerBase {
   // Check if is time to emit a constant pool.
   void CheckConstPool(bool force_emit, bool require_jump);
 
+  RegList registers_mask() const { return registers_mask_; }
+  RegList double_registers_mask() const { return double_registers_mask_; }
+  void set_registers_masks(RegList registers_mask,

[Benedikt Meurer, 2014/03/13 10:41:49]

Nit: One dedicated setter per property.

[vincent.belliard, 2014/03/13 10:53:37]

The aim was to ensure that both masks would always been set at the same time
(and so we never forget to set one). If you prefer two functions, I will change
it.

+                           RegList double_registers_mask) {
+    registers_mask_ = registers_mask;
+    double_registers_mask_ = double_registers_mask;
+  }
+
  protected:
   // Relocation for a type-recording IC has the AST id added to it.  This
   // member variable is a way to pass the information from the call site to
@@ -1559,6 +1567,9 @@ class Assembler : public AssemblerBase {
   // The bound position, before this we cannot do instruction elimination.
   int last_bound_pos_;
 
+  RegList registers_mask_;
+  RegList double_registers_mask_;
+
   // Code emission
   inline void CheckBuffer();
   void GrowBuffer();