julia
d79443c6 - Fix world update after resolving definition effects in `jl_eval_thunk` (#61349)

Commit
40 days ago
Fix world update after resolving definition effects in `jl_eval_thunk` (#61349) PR #60528 removed a world age update after `jl_resolve_definition_effects` as part of a larger refactoring. However `jl_resolve_definition_effects` evaluates selected subexpressions of a thunk - such as the ccall return type - and this may run arbitrary code. Thus a world age update seems required for consistency. IIUC this is the fix requested in the post-commit review by @vtjnash here https://github.com/JuliaLang/julia/pull/60528/files#r2705468506 (Side note - does anyone have an example of code which isn't actively perverse and where this would be required? I think this is good to have regardless (at least as long as we allow arbitrary code to run "before" running the thunk proper) but I couldn't think of an example where this is required to run clean user code.)
Author
Parents
Loading