From 3a560393f1bce659ea675c91150d07ab2a0b01cc Mon Sep 17 00:00:00 2001
From: lolmanurfunny <77128366+lolmanurfunny@users.noreply.github.com>
Date: Fri, 7 Mar 2025 05:45:55 -0500
Subject: [PATCH 1/3] Optimize deletion logic

---
 jecs.luau | 39 ++++++++++++++++++++++-----------------
 1 file changed, 22 insertions(+), 17 deletions(-)

diff --git a/jecs.luau b/jecs.luau
index 9433147..b9c7e05 100644
--- a/jecs.luau
+++ b/jecs.luau
@@ -1113,11 +1113,11 @@ do
 		end
 
 		local delete = entity
-		local component_index = world.component_index
 		local archetypes: Archetypes = world.archetypes
-		local tgt = ECS_PAIR(EcsWildcard, delete)
-		local idr_t = component_index[tgt]
-		local idr = component_index[delete]
+		local component_index = world.component_index
+
+		local idr = component_index[entity]
+		local idr_t = component_index[ECS_PAIR(EcsWildcard, entity)]
 
 		if idr then
 			local flags = idr.flags
@@ -1152,21 +1152,16 @@ do
 
 		if idr_t then
 			for archetype_id in idr_t.columns do
-				local children = {}
 				local idr_t_archetype = archetypes[archetype_id]
-
 				local idr_t_types = idr_t_archetype.types
-
-				for _, child in idr_t_archetype.entities do
-					table.insert(children, child)
-				end
-
+				local children = table.clone(idr_t_archetype.entities)
 				local n = #children
 
 				for _, id in idr_t_types do
 					if not ECS_IS_PAIR(id) then
 						continue
 					end
+
 					local object = ecs_pair_second(world, id)
 					if object == delete then
 						local id_record = component_index[id]
@@ -1181,14 +1176,24 @@ do
 							local on_remove = id_record.hooks.on_remove
 							local to = archetype_traverse_remove(world, id, idr_t_archetype)
 							local empty = #to.types == 0
-							for i = n, 1, -1 do
-								local child = children[i]
-								if on_remove then
+
+							if on_remove then
+								for i = n, 1, -1 do
+									local child = children[i]
 									on_remove(child)
+
+									if not empty then
+										local r = sparse_array[ECS_ENTITY_T_LO(child)]
+										entity_move(entity_index, child, r, to)
+									end
 								end
-								local r = sparse_array[ECS_ENTITY_T_LO(child)]
-								if not empty then
-									entity_move(entity_index, child, r, to)
+							else
+								if not empty then -- no on_remove hook set; fast path
+									for i = n, 1, -1 do
+										local child = children[i]
+										local r = sparse_array[ECS_ENTITY_T_LO(child)]
+										entity_move(entity_index, child, r, to)
+									end
 								end
 							end
 						end

From fa4df24adaba7c815f49cb25f5c043832a75a2b9 Mon Sep 17 00:00:00 2001
From: lolmanurfunny <77128366+lolmanurfunny@users.noreply.github.com>
Date: Fri, 7 Mar 2025 19:56:40 -0500
Subject: [PATCH 2/3] undo variable changes

---
 jecs.luau | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/jecs.luau b/jecs.luau
index b9c7e05..68d6990 100644
--- a/jecs.luau
+++ b/jecs.luau
@@ -1113,11 +1113,11 @@ do
 		end
 
 		local delete = entity
-		local archetypes: Archetypes = world.archetypes
 		local component_index = world.component_index
-
-		local idr = component_index[entity]
-		local idr_t = component_index[ECS_PAIR(EcsWildcard, entity)]
+		local archetypes: Archetypes = world.archetypes
+		local tgt = ECS_PAIR(EcsWildcard, delete)
+		local idr_t = component_index[tgt]
+		local idr = component_index[delete]
 
 		if idr then
 			local flags = idr.flags

From 870823f9fc8290a0886d8f902d20c3e5677f072e Mon Sep 17 00:00:00 2001
From: lolmanurfunny <77128366+lolmanurfunny@users.noreply.github.com>
Date: Fri, 7 Mar 2025 23:17:43 -0500
Subject: [PATCH 3/3] shorten loop

---
 jecs.luau | 22 ++++++----------------
 1 file changed, 6 insertions(+), 16 deletions(-)

diff --git a/jecs.luau b/jecs.luau
index 68d6990..97d1479 100644
--- a/jecs.luau
+++ b/jecs.luau
@@ -1176,24 +1176,14 @@ do
 							local on_remove = id_record.hooks.on_remove
 							local to = archetype_traverse_remove(world, id, idr_t_archetype)
 							local empty = #to.types == 0
-
-							if on_remove then
-								for i = n, 1, -1 do
-									local child = children[i]
+							for i = n, 1, -1 do
+								local child = children[i]
+								if on_remove then
 									on_remove(child)
-
-									if not empty then
-										local r = sparse_array[ECS_ENTITY_T_LO(child)]
-										entity_move(entity_index, child, r, to)
-									end
 								end
-							else
-								if not empty then -- no on_remove hook set; fast path
-									for i = n, 1, -1 do
-										local child = children[i]
-										local r = sparse_array[ECS_ENTITY_T_LO(child)]
-										entity_move(entity_index, child, r, to)
-									end
+								if not empty then
+									local r = sparse_array[ECS_ENTITY_T_LO(child)]
+									entity_move(entity_index, child, r, to)
 								end
 							end
 						end