[orc-rt] Hoist std::decay_t out of helper class.

The helper implementation shouldn't differ based on how it's initialized.

Author: lhames

orc-rt/include/orc-rt/move_only_function.h

@@ -47,7 +47,7 @@ class CallableImpl : public Callable<RetT, ArgTs...> {
   }
 
 private:
-  std::decay_t<CallableT> Callable;
+  CallableT Callable;
 };
 
 } // namespace move_only_function_detail
@@ -66,9 +66,8 @@ class move_only_function<RetT(ArgTs...)> {
 
   template <typename CallableT>
   move_only_function(CallableT &&Callable)
-      : C(std::make_unique<
-            move_only_function_detail::CallableImpl<CallableT, RetT, ArgTs...>>(
-            std::forward<CallableT>(Callable))) {}
+      : C(std::make_unique<move_only_function_detail::CallableImpl<
+              std::decay_t<CallableT>, RetT, ArgTs...>>(std::move(Callable))) {}
 
   RetT operator()(ArgTs... Params) {
     return C->call(std::forward<ArgTs>(Params)...);