Merge pull request #14 from TECHNICANGEL/perf-optimize-entity-manager-369316685402116316

⚡ Avoid redundant vector allocation in EntityManager

Author: TECHNICANGEL

src/EntityManager.h

@@ -9,6 +9,7 @@ class EntityManager {
 private:
     std::vector<std::unique_ptr<Entity>> entities;
     std::unordered_map<std::string, Entity*> entityMap;
+    mutable std::vector<Transform> cachedTransforms;
 
 public:
     // Add entity to the scene
@@ -38,13 +39,13 @@ class EntityManager {
     }
 
     // Get all transforms for acceleration structure
-    std::vector<Transform> getTransforms() const {
-        std::vector<Transform> transforms;
-        transforms.reserve(entities.size());
+    const std::vector<Transform>& getTransforms() const {
+        cachedTransforms.clear();
+        cachedTransforms.reserve(entities.size());
         for (const auto& entity : entities) {
-            transforms.push_back(entity->transform);
+            cachedTransforms.push_back(entity->transform);
         }
-        return transforms;
+        return cachedTransforms;
     }
 
     // Get entity by name

src/main.cpp

@@ -835,7 +835,7 @@ class RacingEngine {
 
             // Prepare TLAS instance data (CPU-side, before command buffer recording)
             if (entityManager.countDynamic() > 0) {
-                auto transforms = entityManager.getTransforms();
+                const auto& transforms = entityManager.getTransforms();
                 acceleration.prepareInstanceData(device, transforms);
             }