Legalize memcpy function signature

Author: bjorn3

cranelift/codegen/src/legalizer/boundary.rs

@@ -772,7 +772,7 @@ pub fn handle_call_abi(
 
     // Start by checking if the argument types already match the signature.
     let sig_ref = match check_call_signature(&pos.func.dfg, inst) {
-        Ok(_) => return spill_call_arguments(pos),
+        Ok(_) => return spill_call_arguments(pos, isa),
         Err(s) => s,
     };
 
@@ -809,7 +809,7 @@ pub fn handle_call_abi(
 
     // Go back and insert spills for any stack arguments.
     pos.goto_inst(inst);
-    spill_call_arguments(pos);
+    spill_call_arguments(pos, isa);
 
     // Yes, we changed stuff.
     true
@@ -1010,7 +1010,7 @@ fn spill_entry_params(func: &mut Function, entry: Block) {
 /// TODO: The outgoing stack slots can be written a bit earlier, as long as there are no branches
 /// or calls between writing the stack slots and the call instruction. Writing the slots earlier
 /// could help reduce register pressure before the call.
-fn spill_call_arguments(pos: &mut FuncCursor) -> bool {
+fn spill_call_arguments(pos: &mut FuncCursor, isa: &dyn TargetIsa) -> bool {
     let inst = pos
         .current_inst()
         .expect("Cursor must point to a call instruction");
@@ -1070,10 +1070,11 @@ fn spill_call_arguments(pos: &mut FuncCursor) -> bool {
             let dest = pos.ins().stack_addr(pointer_type, ss, 0);
             let size = pos.ins().iconst(pointer_type, i64::from(size));
             let signature = {
-                let mut s = Signature::new(CallConv::SystemV /*config.default_call_conv*/);
+                let mut s = Signature::new(isa.default_call_conv());
                 s.params.push(AbiParam::new(pointer_type));
                 s.params.push(AbiParam::new(pointer_type));
                 s.params.push(AbiParam::new(pointer_type));
+                legalize_libcall_signature(&mut s, isa);
                 pos.func.import_signature(s)
             };