commit e256fe100c2d196762f1f87e030f1d2291e0b744
author Ian Rogers <irogers@google.com> Tue Nov 27 13:16:25 2012 -0800
committer Ian Rogers <irogers@google.com> Tue Nov 27 13:16:25 2012 -0800
tree 0e1b8cfc84e290fca5deabd755018c01cd4e0af8
parent 25c15cebebc6e816f27ffc520dd399944fa3f9fe

Avoid direct access to length_.

Should use accessors for fields of any Java heap object. Interestingly
in this case, as the objects are packed GCC for ARM implements the word
load as 4 byte loads, so this code is more optimal.

Change-Id: Iafd3d8ee9f41fb19b3a68b3e90ad51a8acd3d61b

src/object.h

diff --git a/src/object.h b/src/object.h
index efd456e..63228bc 100644
--- a/src/object.h
+++ b/src/object.h
@@ -1110,7 +1110,7 @@
  protected:
   bool IsValidIndex(int32_t index) const
       SHARED_LOCKS_REQUIRED(Locks::mutator_lock_) {
-    if (UNLIKELY(index < 0 || index >= length_)) {
+    if (UNLIKELY(index < 0 || index >= GetLength())) {
       return ThrowArrayIndexOutOfBoundsException(index);
     }
     return true;