From 71422c2b488c8084770d52413c30b436d08dcc91 Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Sat, 27 Jan 2018 15:06:55 -0500
Subject: [PATCH] fuzz_with_unicorn: Move data outside loop

Given we know we're only ever going to use one instruction, we can just presize the vector and reassign to it
instead of potentially reallocating the same memory 100000 times
---
 tests/A64/fuzz_with_unicorn.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/tests/A64/fuzz_with_unicorn.cpp b/tests/A64/fuzz_with_unicorn.cpp
index 60443899..3b122070 100644
--- a/tests/A64/fuzz_with_unicorn.cpp
+++ b/tests/A64/fuzz_with_unicorn.cpp
@@ -96,13 +96,14 @@ static void RunTestInstance(const std::array<u64, 31>& regs, const std::array<Ve
 }
 
 TEST_CASE("A64: Single random instruction", "[a64]") {
+    std::array<u64, 31> regs;
+    std::array<Vector, 32> vecs;
+    std::vector<u32> instructions(1);
+
     for (size_t iteration = 0; iteration < 100000; ++iteration) {
-        std::array<u64, 31> regs;
         std::generate(regs.begin(), regs.end(), []{ return RandInt<u64>(0, ~u64(0)); });
-        std::array<Vector, 32> vecs;
         std::generate(vecs.begin(), vecs.end(), RandomVector);
-        std::vector<u32> instructions;
-        instructions.push_back(GenRandomInst(0, true));
+        instructions[0] = GenRandomInst(0, true);
         u32 pstate = RandInt<u32>(0, 0xF) << 28;
 
         INFO("Instruction: 0x" << std::hex << instructions[0]);