Add bugprone-move-forwarding-reference check to clang-tidy

.clang-tidy

@@ -1,10 +1,12 @@
 ---
 Checks: '-*,
+  bugprone-move-forwarding-reference,
   modernize-deprecated-headers,
   modernize-redundant-void-arg,
   '
 
 WarningsAsErrors: '
+  bugprone-move-forwarding-reference,
   modernize-deprecated-headers,
   modernize-redundant-void-arg,
   '

jlm/rvsdg/reduction-helpers.hpp

@@ -6,10 +6,10 @@
 #ifndef JLM_RVSDG_REDUCTION_HELPERS_HPP
 #define JLM_RVSDG_REDUCTION_HELPERS_HPP
 
-#include <algorithm>
-
 #include <jlm/rvsdg/node.hpp>
 
+#include <algorithm>
+
 namespace jlm::rvsdg
 {
 namespace base
@@ -52,7 +52,7 @@ pairwise_reduce(Container && args, const Reductor & reductor)
 {
   if (args.empty())
   {
-    return std::move(args);
+    return std::forward<Container>(args);
   }
 
   auto left = args.begin();
@@ -75,7 +75,7 @@ pairwise_reduce(Container && args, const Reductor & reductor)
   }
   args.erase(std::next(left), args.end());
 
-  return std::move(args);
+  return std::forward<Container>(args);
 }
 
 /* Test whether any pair of elements of "args" can be reduced according
@@ -137,7 +137,7 @@ commutative_pairwise_reduce(Container && args, const Reductor & reductor)
     ++left;
   }
 
-  return std::move(args);
+  return std::forward<Container>(args);
 }
 
 /* Test whether "flatten_tester" applies to any element of "args". */