diff options
author | Vincent Ambo <mail@tazj.in> | 2022-10-04T15·20+0300 |
---|---|---|
committer | tazjin <tazjin@tvl.su> | 2022-10-06T15·22+0000 |
commit | 44acffc688aa61c363d050e28424041dc9cd0e33 (patch) | |
tree | 92ad69057b93304830d020cc8fadf16c439c4230 /tvix/eval/src/observer.rs | |
parent | c1884919fcfd5f83c54f7729a3b5c347c22fdd31 (diff) |
refactor(tvix/eval): builtins now contain closures r/5040
For some upcoming builtins (notably, import) we need to capture arguments in the builtin's implementation. To allow this, we can no longer use function pointers for builtins, but must use a reference-counted closure object instead. Unfortunately this adds an extra pointer operation to every builtin call. We should benchmark this later against having a split builtin representation. Change-Id: I109d98d0e25998870542f47573eb1ec2e546f2a2 Reviewed-on: https://cl.tvl.fyi/c/depot/+/6856 Tested-by: BuildkiteCI Reviewed-by: grfn <grfn@gws.fyi>
Diffstat (limited to 'tvix/eval/src/observer.rs')
0 files changed, 0 insertions, 0 deletions