Skip to content

Commit

Permalink
Check both use_ccall_macro and wrap_variadic_function for the cod…
Browse files Browse the repository at this point in the history
…egen of `FunctionVariadic`

cc @melonedo
  • Loading branch information
Gnimuc authored Aug 9, 2021
1 parent fea7f3f commit 50670a8
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/generator/codegen.jl
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,8 @@ end
function emit!(dag::ExprDAG, node::ExprNode{FunctionVariadic}, options::Dict; args...)
# @ccall is needed to support variadic argument
use_ccall_macro = get(options, "use_ccall_macro", true)
if use_ccall_macro
wrap_variadic_function = get(options, "wrap_variadic_function", false)
if use_ccall_macro && wrap_variadic_function
cursor = node.cursor
is_strict_typed = get(options, "is_function_strictly_typed", false)
arg_names, args = _get_func_arg(cursor, options, dag)
Expand Down

1 comment on commit 50670a8

@Gnimuc
Copy link
Member Author

@Gnimuc Gnimuc commented on 50670a8 Aug 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an oversight of #315

Please sign in to comment.