Fix some issue with RT API UnorderedMap

Fix some issue with RT API UnorderedMap

Author: jaladreips

IGC/AdaptorCommon/RayTracing/API/ADT/Array.h

@@ -114,7 +114,7 @@ template <typename T> class Array {
     End = Start + NumElts;
 
     for (uint32_t i = 0; i < NumElts; i++)
-      Start[i] = typename iterator::value_type();
+      new (&Start[i]) typename iterator::value_type;
   }
 
   Array &operator=(const Array &) = delete;

IGC/AdaptorCommon/RayTracing/API/ADT/UnorderedMap.h

@@ -94,19 +94,24 @@ class UnorderedMap {
     if (idx < Keys.size()) {
       Values[idx] = std::move(value);
     } else {
-      // Grow arrays by 1
-      Array<KeyT> newKeys(Keys.size() + 1);
-      Array<ValueT> newValues(Values.size() + 1);
-      for (size_t i = 0; i < Keys.size(); ++i) {
-        newKeys[i] = Keys[i];
-        newValues[i] = std::move(Values[i]);
+      uint32_t currSize = Keys.size();
+
+      Array<KeyT> oldKeys = std::move(Keys);
+      Array<ValueT> oldValues = std::move(Values);
+
+      Keys = Array<KeyT>(currSize + 1);
+      Values = Array<ValueT>(currSize + 1);
+
+      for (uint32_t i = 0; i < currSize; ++i) {
+        Keys[i] = std::move(oldKeys[i]);
+        Values[i] = std::move(oldValues[i]);
       }
-      newKeys[Keys.size()] = key;
-      newValues[Values.size()] = std::move(value);
-      Keys.destroy();
-      Values.destroy();
-      Keys = std::move(newKeys);
-      Values = std::move(newValues);
+
+      Keys[currSize] = key;
+      Values[currSize] = std::move(value);
+
+      oldKeys.destroy();
+      oldValues.destroy();
     }
   }