Reviewers: Yang,

Message:
Just a little drive-by change.

Description:
Remove some obsolete GC-safe accessors.

[email protected]
BUG=v8:1490


Please review this at http://codereview.chromium.org/8142023/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/objects-inl.h


Index: src/objects-inl.h
diff --git a/src/objects-inl.h b/src/objects-inl.h
index 11b73d383582c1ebe525f94986df05495135ec20..6509f136caede0ac754f6053ae0a91ce67578933 100644
--- a/src/objects-inl.h
+++ b/src/objects-inl.h
@@ -88,15 +88,6 @@ PropertyDetails PropertyDetails::AsDeleted() {
   }


-// GC-safe accessors do not use HeapObject::GetHeap(), but access TLS instead.
-#define ACCESSORS_GCSAFE(holder, name, type, offset)                    \
-  type* holder::name() { return type::cast(READ_FIELD(this, offset)); } \
-  void holder::set_##name(type* value, WriteBarrierMode mode) {         \
-    WRITE_FIELD(this, offset, value);                                   \
-    CONDITIONAL_WRITE_BARRIER(HEAP, this, offset, value, mode);         \
-  }
-
-
 #define SMI_ACCESSORS(holder, name, offset)             \
   int holder::name() {                                  \
     Object* value = READ_FIELD(this, offset);           \
@@ -3414,8 +3405,8 @@ ACCESSORS(BreakPointInfo, break_point_objects, Object, kBreakPointObjectsIndex)
 #endif

 ACCESSORS(SharedFunctionInfo, name, Object, kNameOffset)
-ACCESSORS_GCSAFE(SharedFunctionInfo, construct_stub, Code, kConstructStubOffset) -ACCESSORS_GCSAFE(SharedFunctionInfo, initial_map, Object, kInitialMapOffset)
+ACCESSORS(SharedFunctionInfo, construct_stub, Code, kConstructStubOffset)
+ACCESSORS(SharedFunctionInfo, initial_map, Object, kInitialMapOffset)
 ACCESSORS(SharedFunctionInfo, instance_class_name, Object,
           kInstanceClassNameOffset)
 ACCESSORS(SharedFunctionInfo, function_data, Object, kFunctionDataOffset)
@@ -3888,7 +3879,7 @@ void JSProxy::InitializeBody(int object_size, Object* value) {


 ACCESSORS(JSWeakMap, table, Object, kTableOffset)
-ACCESSORS_GCSAFE(JSWeakMap, next, Object, kNextOffset)
+ACCESSORS(JSWeakMap, next, Object, kNextOffset)


 ObjectHashTable* JSWeakMap::unchecked_table() {


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to