Move constructor has priority over copyable

Author: lucic71

cpp2rust/converter/converter.cpp

@@ -4617,9 +4617,6 @@ std::string Converter::ConvertPlaceholder(clang::Expr *expr, clang::Expr *arg,
     }
     if (auto *record = arg->getType()->getAsCXXRecordDecl();
         record && IsUserDefinedDecl(record)) {
-      if (TypeIsCopyable(arg->getType())) {
-        return ConvertRValue(arg);
-      }
       for (auto *ctor : record->ctors()) {
         if (!IsUserDefinedMoveConstructor(ctor)) {
           continue;
@@ -4628,6 +4625,9 @@ std::string Converter::ConvertPlaceholder(clang::Expr *expr, clang::Expr *arg,
         Convert(MakeConstructExpr(ctx_, arg->getType(), ctor, arg));
         return std::move(buf).str();
       }
+      if (TypeIsCopyable(arg->getType())) {
+        return ConvertRValue(arg);
+      }
       return ConvertFreshRValue(arg);
     }
     return std::format("std::mem::take(&mut {})", ConvertLValue(arg));