mirror of https://github.com/m-labs/artiq.git
transforms.llvm_ir_generator: use sret attribute on FFI calls too.
This commit is contained in:
parent
7f914a057c
commit
d436093317
|
@ -940,10 +940,17 @@ class LLVMIRGenerator:
|
||||||
llfunname = insn.target_function().type.name
|
llfunname = insn.target_function().type.name
|
||||||
llfun = self.llmodule.get_global(llfunname)
|
llfun = self.llmodule.get_global(llfunname)
|
||||||
if llfun is None:
|
if llfun is None:
|
||||||
llfunty = ll.FunctionType(self.llty_of_type(insn.type, for_return=True),
|
llretty = self.llty_of_type(insn.type, for_return=True)
|
||||||
|
if self.needs_sret(llretty):
|
||||||
|
llfunty = ll.FunctionType(llvoid, [llretty.as_pointer()] +
|
||||||
[llarg.type for llarg in llargs])
|
[llarg.type for llarg in llargs])
|
||||||
|
else:
|
||||||
|
llfunty = ll.FunctionType(llretty, [llarg.type for llarg in llargs])
|
||||||
|
|
||||||
llfun = ll.Function(self.llmodule, llfunty,
|
llfun = ll.Function(self.llmodule, llfunty,
|
||||||
insn.target_function().type.name)
|
insn.target_function().type.name)
|
||||||
|
if self.needs_sret(llretty):
|
||||||
|
llfun.args[0].add_attribute('sret')
|
||||||
return llfun, list(llargs)
|
return llfun, list(llargs)
|
||||||
|
|
||||||
# See session.c:{send,receive}_rpc_value and comm_generic.py:_{send,receive}_rpc_value.
|
# See session.c:{send,receive}_rpc_value and comm_generic.py:_{send,receive}_rpc_value.
|
||||||
|
@ -1087,8 +1094,6 @@ class LLVMIRGenerator:
|
||||||
llnormalblock=None, llunwindblock=None)
|
llnormalblock=None, llunwindblock=None)
|
||||||
elif types.is_c_function(insn.target_function().type):
|
elif types.is_c_function(insn.target_function().type):
|
||||||
llfun, llargs = self._prepare_ffi_call(insn)
|
llfun, llargs = self._prepare_ffi_call(insn)
|
||||||
return self.llbuilder.call(llfun, llargs,
|
|
||||||
name=insn.name)
|
|
||||||
else:
|
else:
|
||||||
llfun, llargs = self._prepare_closure_call(insn)
|
llfun, llargs = self._prepare_closure_call(insn)
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue