coop/rose-ash
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

JIT: don't mutate l_compiled when skipping warned closures

Browse Source 
Setting jit_failed_sentinel on new instances of warned function names
caused hanging — the mutation interfered with CEK execution. Now just
return None without touching l_compiled. The _jit_warned hash lookup
is a cheap check per call with no side effects.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
giles
2026-03-29 19:57:31 +00:00
parent 62d8602de4
commit f978792e62
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
hosts/ocaml/bin/sx_server.ml
View File

@@ -728,8 +728,7 @@ let register_jit_hook env =
| Some _ -> None (* compile failed or disabled — CEK handles *) | Some _ -> None (* compile failed or disabled — CEK handles *)
| None -> | None ->
if !_jit_compiling then None if !_jit_compiling then None
else if Hashtbl.mem _jit_warned fn_name then else if Hashtbl.mem _jit_warned fn_name then None
(l.l_compiled <- Some Sx_vm.jit_failed_sentinel; None)
else begin else begin
_jit_compiling := true; _jit_compiling := true;
let t0 = Unix.gettimeofday () in let t0 = Unix.gettimeofday () in