about summary refs log tree commit diff
path: root/rustfmt.toml
diff options
context:
space:
mode:
authorVincent Ambo <mail@tazj.in>2022-09-02T18·49+0300
committertazjin <tazjin@tvl.su>2022-09-08T12·53+0000
commit2246a31e726762ea741a299f598c7878fa66dd83 (patch)
treea3d44e78f5f6cef449c0bb5247cc82dd62fffaa1 /rustfmt.toml
parentcc526a2c873524faa83cad62bde2edda59ea7820 (diff)
refactor(tvix/eval): return call frame result from VM::call r/4748
Previously, "calling" (setting up the VM run loop for executing a call
frame) and "running" (running this loop to completion) were separate
operations.

This was basically an attempt to avoid nesting `VM::run` invocations.
However, doing things this way introduced some tricky bugs for exiting
out of the call frames of thunks vs. builtins & closures.

For now, we unify the two operations and always return the value to
the caller directly. For now this makes calls a little less effective,
but it gives us a chance to nail down some other strange behaviours
and then re-optimise this afterwards.

To make sure we tackle this again further down I've added it to the
list of known possible optimisations.

Change-Id: I96828ab6a628136e0bac1bf03555faa4e6b74ece
Reviewed-on: https://cl.tvl.fyi/c/depot/+/6415
Reviewed-by: sterni <sternenseemann@systemli.org>
Tested-by: BuildkiteCI
Diffstat (limited to 'rustfmt.toml')
0 files changed, 0 insertions, 0 deletions