From e6ec9e48b4c1a122de813ba2084c3d489ebc1f39 Mon Sep 17 00:00:00 2001 From: srikappa-amzn Date: Fri, 11 Jun 2021 16:17:26 -0700 Subject: [PATCH] Changed spawnables benchmark test to create multiple spawnables at once --- .../Benchmark/SpawnableCreateBenchmarks.cpp | 20 ++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/Code/Framework/AzToolsFramework/Tests/Prefab/Benchmark/SpawnableCreateBenchmarks.cpp b/Code/Framework/AzToolsFramework/Tests/Prefab/Benchmark/SpawnableCreateBenchmarks.cpp index fb8a9a389c..bf04f15cce 100644 --- a/Code/Framework/AzToolsFramework/Tests/Prefab/Benchmark/SpawnableCreateBenchmarks.cpp +++ b/Code/Framework/AzToolsFramework/Tests/Prefab/Benchmark/SpawnableCreateBenchmarks.cpp @@ -29,26 +29,28 @@ namespace Benchmark {}, m_pathString)); + AZStd::vector> spawnables; + spawnables.resize(numSpawnables); + auto& prefabDom = m_prefabSystemComponent->FindTemplateDom(instance->GetTemplateId()); for (auto _ : state) { - state.PauseTiming(); - - AzFramework::Spawnable spawnable; - AzToolsFramework::Prefab::SpawnableUtils::CreateSpawnable(spawnable, prefabDom); - - - state.ResumeTiming(); + for (int instanceCounter = 0; instanceCounter < numSpawnables; ++instanceCounter) + { + AZStd::unique_ptr spawnable = AZStd::make_unique(); + AzToolsFramework::Prefab::SpawnableUtils::CreateSpawnable(*spawnable, prefabDom); + spawnables[instanceCounter] = AZStd::move(spawnable); + } } state.SetComplexityN(numSpawnables); } - /* BENCHMARK_REGISTER_F(BM_SpawnableCreate, CreateSpawnable_SingleEntityInstance) ->RangeMultiplier(10) ->Range(100, 10000) ->Unit(benchmark::kMillisecond) - ->Complexity();*/ + ->Complexity(); } #endif +