Use move helper for rdcarray move append

* This makes a big difference for move-appending basic types like bytes or ints

renderdoc/api/replay/rdcarray.h

@@ -568,8 +568,8 @@ public:
   inline void append(rdcarray<T> &&in)
   {
     reserve(size() + in.size());
-    for(size_t i = 0; i < in.size(); i++)
-      push_back(std::move(in[i]));
+    ItemCopyHelper<T>::moveRange(data() + size(), in.data(), in.size());
+    setUsedCount(usedCount + in.size());
     // don't have to clear here, since moved object can be left in any indeterminate but valid state
     // (an all the members are in that state, while the array is fully valid), but this gives fewer
     // surprises.

renderdoc/replay/basic_types_tests.cpp

@@ -415,6 +415,15 @@ static void TestBasic()
   CHECK(test.size() == 7);
   CHECK(x == 10);
   CHECK(test[2] == 6);
+
+  rdcarray<inner> other = {1, 2, 3};
+
+  test.append(other);
+
+  CHECK(test.size() == 10);
+  CHECK(test[7] == 1);
+  CHECK(test[8] == 2);
+  CHECK(test[9] == 3);
 }
 
 static int32_t constructor = 0;
@@ -1307,6 +1316,32 @@ TEST_CASE("Test array type", "[basictypes]")
     CHECK(movedDestructor == 17);
     CHECK(copyAssignment == 0);
     CHECK(moveAssignment == 0);
+
+    // move append
+    rdcarray<ConstructorCounter> other = {1, 2, 3};
+
+    CHECK(valueConstructor == 8);
+    CHECK(copyConstructor == 5);
+    CHECK(moveConstructor == 21);
+    CHECK(destructor == 21);
+    CHECK(movedDestructor == 17);
+    CHECK(copyAssignment == 0);
+    CHECK(moveAssignment == 0);
+
+    test.append(std::move(other));
+
+    CHECK(valueConstructor == 8);
+    CHECK(copyConstructor == 5);
+    CHECK(moveConstructor == 24);
+    CHECK(destructor == 24);
+    CHECK(movedDestructor == 20);
+    CHECK(copyAssignment == 0);
+    CHECK(moveAssignment == 0);
+
+    CHECK(test.size() == 13);
+    CHECK(test[10].value == 1);
+    CHECK(test[11].value == 2);
+    CHECK(test[12].value == 3);
   };
 };