]> gitweb.factorcode.org Git - factor.git/blobdiff - vm/objects.cpp
webapps.wiki: adding search bar
[factor.git] / vm / objects.cpp
index 549b5cbaef98fa6ee477ed49d04d429236a72258..bc858994657333a6aa88aebd31e4bb5e8d847557 100644 (file)
@@ -2,6 +2,13 @@
 
 namespace factor {
 
+// Size of the object pointed to by a tagged pointer
+cell object_size(cell tagged) {
+  if (immediate_p(tagged))
+    return 0;
+  return untag<object>(tagged)->size();
+}
+
 void factor_vm::primitive_special_object() {
   fixnum n = untag_fixnum(ctx->peek());
   ctx->replace(special_objects[n]);
@@ -19,16 +26,12 @@ void factor_vm::primitive_identity_hashcode() {
   ctx->replace(tag_fixnum(obj->hashcode()));
 }
 
-void factor_vm::compute_identity_hashcode(object* obj) {
+void factor_vm::primitive_compute_identity_hashcode() {
+  object* obj = untag<object>(ctx->pop());
   object_counter++;
   if (object_counter == 0)
     object_counter++;
-  obj->set_hashcode((cell) obj ^ object_counter);
-}
-
-void factor_vm::primitive_compute_identity_hashcode() {
-  object* obj = untag<object>(ctx->pop());
-  compute_identity_hashcode(obj);
+  obj->set_hashcode((cell)obj ^ object_counter);
 }
 
 void factor_vm::primitive_set_slot() {
@@ -41,87 +44,44 @@ void factor_vm::primitive_set_slot() {
   write_barrier(slot_ptr);
 }
 
-/* Allocates memory */
-cell factor_vm::clone_object(cell obj_) {
-  data_root<object> obj(obj_, this);
+// Allocates memory
+void factor_vm::primitive_clone() {
 
-  if (immediate_p(obj.value()))
-    return obj.value();
-  else {
-    cell size = object_size(obj.value());
-    object* new_obj = allot_object(obj.type(), size);
-    memcpy(new_obj, obj.untagged(), size);
-    new_obj->set_hashcode(0);
-    return tag_dynamic(new_obj);
-  }
-}
+  data_root<object> obj(ctx->peek(), this);
 
-/* Allocates memory */
-void factor_vm::primitive_clone() { ctx->replace(clone_object(ctx->peek())); }
-
-/* Size of the object pointed to by a tagged pointer */
-cell factor_vm::object_size(cell tagged) {
-  if (immediate_p(tagged))
-    return 0;
-  else
-    return untag<object>(tagged)->size();
+  if (immediate_p(obj.value()))
+    return;
+  cell size = object_size(obj.value());
+  object* new_obj = allot_object(obj.type(), size);
+  memcpy(new_obj, obj.untagged(), size);
+  new_obj->set_hashcode(0);
+  ctx->replace(tag_dynamic(new_obj));
 }
 
-/* Allocates memory */
+// Allocates memory
 void factor_vm::primitive_size() {
-  ctx->push(from_unsigned_cell(object_size(ctx->pop())));
+  ctx->replace(from_unsigned_cell(object_size(ctx->peek())));
 }
 
 struct slot_become_fixup : no_fixup {
   std::map<object*, object*>* become_map;
 
-  slot_become_fixup(std::map<object*, object*>* become_map_)
-      : become_map(become_map_) {}
+  slot_become_fixup(std::map<object*, object*>* become_map)
+      : become_map(become_map) {}
 
   object* fixup_data(object* old) {
     std::map<object*, object*>::const_iterator iter = become_map->find(old);
     if (iter != become_map->end())
       return iter->second;
-    else
-      return old;
+    return old;
   }
 };
 
-struct object_become_visitor {
-  slot_visitor<slot_become_fixup>* workhorse;
-
-  explicit object_become_visitor(slot_visitor<slot_become_fixup>* workhorse_)
-      : workhorse(workhorse_) {}
-
-  void operator()(object* obj) { workhorse->visit_slots(obj); }
-};
-
-struct code_block_become_visitor {
-  slot_visitor<slot_become_fixup>* workhorse;
-
-  explicit code_block_become_visitor(
-      slot_visitor<slot_become_fixup>* workhorse_)
-      : workhorse(workhorse_) {}
-
-  void operator()(code_block* compiled, cell size) {
-    workhorse->visit_code_block_objects(compiled);
-    workhorse->visit_embedded_literals(compiled);
-  }
-};
-
-struct code_block_write_barrier_visitor {
-  code_heap* code;
-
-  explicit code_block_write_barrier_visitor(code_heap* code_) : code(code_) {}
-
-  void operator()(code_block* compiled, cell size) {
-    code->write_barrier(compiled);
-  }
-};
-
-/* classes.tuple uses this to reshape tuples; tools.deploy.shaker uses this
-   to coalesce equal but distinct quotations and wrappers. */
+// classes.tuple uses this to reshape tuples; tools.deploy.shaker uses this
+// to coalesce equal but distinct quotations and wrappers.
+// Calls gc
 void factor_vm::primitive_become() {
+  primitive_minor_gc();
   array* new_objects = untag_check<array>(ctx->pop());
   array* old_objects = untag_check<array>(ctx->pop());
 
@@ -129,39 +89,39 @@ void factor_vm::primitive_become() {
   if (capacity != array_capacity(old_objects))
     critical_error("bad parameters to become", 0);
 
-  /* Build the forwarding map */
+  // Build the forwarding map
   std::map<object*, object*> become_map;
 
   for (cell i = 0; i < capacity; i++) {
-    tagged<object> old_obj(array_nth(old_objects, i));
-    tagged<object> new_obj(array_nth(new_objects, i));
-
-    if (old_obj != new_obj)
-      become_map[old_obj.untagged()] = new_obj.untagged();
+    cell old_ptr = array_nth(old_objects, i);
+    cell new_ptr = array_nth(new_objects, i);
+    if (old_ptr != new_ptr)
+      become_map[untag<object>(old_ptr)] = untag<object>(new_ptr);
   }
 
-  /* Update all references to old objects to point to new objects */
+  // Update all references to old objects to point to new objects
   {
-    slot_visitor<slot_become_fixup> workhorse(this,
-                                              slot_become_fixup(&become_map));
-    workhorse.visit_roots();
-    workhorse.visit_contexts();
-
-    object_become_visitor object_visitor(&workhorse);
-    each_object(object_visitor);
-
-    code_block_become_visitor code_block_visitor(&workhorse);
-    each_code_block(code_block_visitor);
+    slot_visitor<slot_become_fixup> visitor(this,
+                                            slot_become_fixup(&become_map));
+    visitor.visit_all_roots();
+
+    auto object_become_func = [&](object* obj) {
+      visitor.visit_slots(obj);
+    };
+    each_object(object_become_func);
+
+    auto code_block_become_func = [&](code_block* compiled, cell size) {
+      (void)size;
+      visitor.visit_code_block_objects(compiled);
+      visitor.visit_embedded_literals(compiled);
+      code->write_barrier(compiled);
+    };
+    each_code_block(code_block_become_func);
   }
 
-  /* Since we may have introduced old->new references, need to revisit
-     all objects and code blocks on a minor GC. */
+  // Since we may have introduced old->new references, need to revisit
+  // all objects and code blocks on a minor GC.
   data->mark_all_cards();
-
-  {
-    code_block_write_barrier_visitor code_block_visitor(code);
-    each_code_block(code_block_visitor);
-  }
 }
 
 }