Skip to content

Workaround for bug with caml_sys_argv#5

Open
lthls wants to merge 1 commit intobvaugon:masterfrom
lthls:patch-1
Open

Workaround for bug with caml_sys_argv#5
lthls wants to merge 1 commit intobvaugon:masterfrom
lthls:patch-1

Conversation

@lthls
Copy link
Copy Markdown

@lthls lthls commented Aug 17, 2021

There's a hack in the compiler that miscompiles the caml_sys_argv primitive if it's declared as an external directly (it's supposed to be exposed through the "%sys_argv" builtin primitive, not directly).
I'm not sure exactly why it never was an issue before, but I believe that without this patch trying to call the caml_sys_argv primitive through the generated Prim module will cause a segmentation fault.

There's a hack in the compiler that miscompiles the `caml_sys_argv` primitive if it's declared as an external directly (it's supposed to be exposed through the `"%sys_argv"` builtin primitive, not directly).
I'm not sure exactly why it never was an issue before, but I believe that without this patch trying to call the `caml_sys_argv` primitive through the generated `Prim` module will cause a segmentation fault.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant