commit 5d76c435082332ef79a22962386fa92a0870e378
author Ian Rogers <irogers@google.com> Mon Oct 31 21:42:49 2011 -0700
committer Ian Rogers <irogers@google.com> Wed Nov 02 01:25:57 2011 -0700
tree db8fdd7bfba3617494157b483e9df979f76dfdcc
parent 4b6fe5a568ca2bc5e8fa110bf7af692cab220a15

Mark non-image spaces and use write barrier for image spaces.

Don't mark string and class roots that are in the image, alloc space
references will be caught by the write barrier.

Change-Id: Idcf9e4ede3b83556d4f8a01276273726dc6eea46

src/object.cc

diff --git a/src/object.cc b/src/object.cc
index 45fc781..2bb6d2d 100644
--- a/src/object.cc
+++ b/src/object.cc
@@ -127,11 +127,13 @@
   return descriptor;
 }
 
-Class* Field::GetType() const {
-  if (type_ == NULL) {
-    type_ = Runtime::Current()->GetClassLinker()->ResolveType(GetTypeIdx(), this);
+Class* Field::GetType() {
+  Class* type = GetFieldObject<Class*>(OFFSET_OF_OBJECT_MEMBER(Field, type_), false);
+  if (type == NULL) {
+    type = Runtime::Current()->GetClassLinker()->ResolveType(GetTypeIdx(), this);
+    SetFieldObject(OFFSET_OF_OBJECT_MEMBER(Field, type_), type, false);
   }
-  return type_;
+  return type;
 }
 
 void Field::InitJavaFields() {
@@ -376,8 +378,11 @@
   DCHECK_EQ(*p, ')');
   ++p;
 
-  java_parameter_types_ = parameters;
-  java_return_type_ = ExtractNextClassFromSignature(class_linker, cl, p);
+  SetFieldObject(OFFSET_OF_OBJECT_MEMBER(Method, java_parameter_types_),
+                 parameters, false);
+  Class* java_return_type = ExtractNextClassFromSignature(class_linker, cl, p);
+  SetFieldObject(OFFSET_OF_OBJECT_MEMBER(Method, java_return_type_),
+                 java_return_type, false);
 }
 
 void Method::InitJavaFields() {