clang-tidy: Remove unnecessary std::move calls

Change-Id: Ic5a26cfa7c2cb1f993ea4e8ccb30c4a093a75f42
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
diff --git a/test/openpower-pels/pel_test.cpp b/test/openpower-pels/pel_test.cpp
index 67ce75f..c5903c3 100644
--- a/test/openpower-pels/pel_test.cpp
+++ b/test/openpower-pels/pel_test.cpp
@@ -761,11 +761,11 @@
     // This will be trimmed when added
     std::vector<uint8_t> hugeCustomData(17000, 0x42);
 
-    ffdc.emplace_back(std::move(getJSONFFDC(dir)));
-    ffdc.emplace_back(std::move(getCBORFFDC(dir)));
-    ffdc.emplace_back(std::move(getTextFFDC(dir)));
-    ffdc.emplace_back(std::move(getCustomFFDC(dir, customData)));
-    ffdc.emplace_back(std::move(getCustomFFDC(dir, hugeCustomData)));
+    ffdc.emplace_back(getJSONFFDC(dir));
+    ffdc.emplace_back(getCBORFFDC(dir));
+    ffdc.emplace_back(getTextFFDC(dir));
+    ffdc.emplace_back(getCustomFFDC(dir, customData));
+    ffdc.emplace_back(getCustomFFDC(dir, hugeCustomData));
 
     PEL pel{regEntry, 42,   timestamp, phosphor::logging::Entry::Level::Error,
             ad,       ffdc, dataIface, journal};