commit 19e53eb4d48c4b90ebe6535295ccec9c225d62c6
author Shih-wei Liao <sliao@google.com> Wed Oct 05 19:11:46 2011 -0700
committer Shih-wei Liao <sliao@google.com> Wed Oct 05 23:24:21 2011 -0700
tree f256433c98b5b0bac344ef1727d2cdcf707cd615
parent 8c2f6414e1cff9a024a14369352f7ce112528b9e

Callee-save these ARM registers: R5-R8, R10-R11 and LR.

To enable compiler's register promotion.

Change-Id: Ic186bf735af686c667fb393c64dc17bae0733703

src/calling_convention_arm.cc

diff --git a/src/calling_convention_arm.cc b/src/calling_convention_arm.cc
index 0a5868b..e4f7b19 100644
--- a/src/calling_convention_arm.cc
+++ b/src/calling_convention_arm.cc
@@ -111,42 +111,19 @@
     }
   }
   padding_ = padding;
-  if (method->IsSynchronized()) {
-    // Preserve callee saves that may be clobbered during monitor enter where
-    // we copy across R0 to R3
-    if (method->NumArgs() > 0) {
-      callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R4));
-      if (method->NumArgs() > 1) {
-        callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R5));
-        if (method->NumArgs() > 2) {
-          callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R6));
-          if (method->NumArgs() > 3) {
-            callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R7));
-          }
-        }
-      }
-    }
-  }
+
+  callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R5));
+  callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R6));
+  callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R7));
+  callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R8));
+  callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R10));
+  callee_save_regs_.push_back(ArmManagedRegister::FromCoreRegister(R11));
 }
 
 uint32_t ArmJniCallingConvention::CoreSpillMask() const {
   // Compute spill mask to agree with callee saves initialized in the constructor
   uint32_t result = 0;
-  Method* method = GetMethod();
-  if (method->IsSynchronized()) {
-    if (method->NumArgs() > 0) {
-      result |= 1 << R4;
-      if (method->NumArgs() > 1) {
-        result |= 1 << R5;
-        if (method->NumArgs() > 2) {
-          result |= 1 << R6;
-          if (method->NumArgs() > 3) {
-            result |= 1 << R7;
-          }
-        }
-      }
-    }
-  }
+  result =  1 << R5 | 1 << R6 | 1 << R7 | 1 << R8 | 1 << R10 | 1 << R11 | 1 << LR;
   return result;
 }