ArkScript-lang / Ark / 20177078200

93%

Build Type

push

github

Committed by SuperFola

Commit Message

fix(ir optimizer): do not try to optimize calls to builtins outside functions, otherwise this messes up the stack

Somehow the optimizer figured that
(let sample ...)
(print sample)
Could be optimized with a single CALL_BUILTIN_WITHOUT_RETURN_ADDRESS, because it was a sequence of load, store,
push return address, load symbol by index, builtin, call, but it didn't see that it wasn't in a function definition,
which is the only place where the whole body of a function can be replaced by a single instruction.

Run Details

25 of 25 new or added lines in 2 files covered. (100.0%)

9 existing lines in 1 file now uncovered.

8094 of 8931 relevant lines covered (90.63%)

196834.79 hits per line