about summary refs log tree commit diff
path: root/users/wpcarro/scratch
diff options
context:
space:
mode:
authorsterni <sternenseemann@systemli.org>2022-10-15T13·52+0200
committerclbot <clbot@tvl.fyi>2022-10-15T14·12+0000
commitf3c27f1717f0b92b05340c0bc6ba44007ce9d90b (patch)
tree989566dc26387d1e7849df4de0bdc841cdf963aa /users/wpcarro/scratch
parentcaf9cbf71147054c01c6fdf49773d646cbfb46eb (diff)
fix(tvix/eval): bring foldl' strictness in line with C++ Nix r/5133
Working on https://github.com/NixOS/nix/pull/7158, I discovered that C++
Nix actually is strict in the accumulator, just not in the first value.
This seems due to the fact that in the C++ evaluator, function calls
don't seem to be thunked unconditionally and foldl' just elects not to
wrap it in a thunk (don't quote me on this summary, even though it seems
to line up with the code for primop_foldlStrict and testable behavior).

It doesn't seem worth it to risk breaking the odd Nix expression just to
be strict in one more value per invocation of foldl' (i.e. the initial
accumulator value `nul`), so let's match the existing C++ Nix behavior
here.

Change-Id: If59e62271a90d97cb440f0ca72a58ec7840d1690
Reviewed-on: https://cl.tvl.fyi/c/depot/+/7022
Autosubmit: sterni <sternenseemann@systemli.org>
Tested-by: BuildkiteCI
Reviewed-by: tazjin <tazjin@tvl.su>
Diffstat (limited to 'users/wpcarro/scratch')
0 files changed, 0 insertions, 0 deletions