22278: Reduces transactional load time (#308)

howsoai · Nov 20, 2024 · 3f9a2f0 · 3f9a2f0
1 parent b334ac2
commit 3f9a2f0
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 15 deletions.
diff --git a/src/Amalgam/Parser.cpp b/src/Amalgam/Parser.cpp
@@ -121,6 +121,8 @@ std::tuple<EvaluableNodeReference, std::vector<std::string>, size_t> Parser::Par
 {
 	preevaluationNodes.clear();
 	parentNodes.clear();
+	//since the type of parentNodes can be expensive to iterate over, reduce the size
+	parentNodes.shrink_to_fit();
 
 	EvaluableNode *top_node = ParseCode();
 

diff --git a/src/Amalgam/evaluablenode/EvaluableNodeManagement.cpp b/src/Amalgam/evaluablenode/EvaluableNodeManagement.cpp
@@ -11,12 +11,6 @@
 Concurrency::ReadWriteMutex EvaluableNodeManager::memoryModificationMutex;
 #endif
 
-
-#if defined(MULTITHREAD_SUPPORT) || defined(MULTITHREAD_INTERFACE)
-thread_local
-#endif
-EvaluableNode::ReferenceAssocType EvaluableNodeManager::nodeToParentNodeCache;
-
 const double EvaluableNodeManager::allocExpansionFactor = 1.5;
 
 EvaluableNodeManager::~EvaluableNodeManager()

diff --git a/src/Amalgam/evaluablenode/EvaluableNodeManagement.h b/src/Amalgam/evaluablenode/EvaluableNodeManagement.h
@@ -658,8 +658,8 @@ class EvaluableNodeManager
 		if(tree == nullptr)
 			return;
 
-		nodeToParentNodeCache.clear();
-		UpdateFlagsForNodeTreeRecurse(tree, nullptr, nodeToParentNodeCache);
+		EvaluableNode::ReferenceAssocType node_to_parent_cache;
+		UpdateFlagsForNodeTreeRecurse(tree, nullptr, node_to_parent_cache);
 	}
 
 	//updates idempotency flags for tree and returns true if tree is idempotent
@@ -1105,13 +1105,6 @@ class EvaluableNodeManager
 	//only allocated if needed
 	std::unique_ptr<NodesReferenced> nodesCurrentlyReferenced;
 
-	//buffer used for updating EvaluableNodeFlags, particularly UpdateFlagsForNodeTree
-#if defined(MULTITHREAD_SUPPORT) || defined(MULTITHREAD_INTERFACE)
-	thread_local
-#endif
-		static EvaluableNode::ReferenceAssocType nodeToParentNodeCache;
-
-
 	//extra space to allocate when allocating
 	static const double allocExpansionFactor;
 };