diff options
| author | sterni <sternenseemann@systemli.org> | 2023-06-02T20·38+0200 |
|---|---|---|
| committer | clbot <clbot@tvl.fyi> | 2023-06-07T15·17+0000 |
| commit | 10c6cb7251480ca12e67b3d237740e6dcb93f87e (patch) | |
| tree | f7e12f8cad3e3a8cc2f981f5005e37fd4ad076ef /tvix/eval/src/opcode.rs | |
| parent | 617130b08818aba0db5836ce9a71adae4991fb0a (diff) | |
fix(tvix/eval): type check function argument with set pattern r/6243
C++ Nix forces and typechecks the passed argument even if it is not necessary in order to compute the return value of the function. I discovered this when I thought our formals miscompilation might be that we are too strict, but doesn't look like it in this case. Change-Id: Ifb3c92592293052c489d1e3ae8c7c54e4b6b4dc6 Reviewed-on: https://cl.tvl.fyi/c/depot/+/8701 Tested-by: BuildkiteCI Autosubmit: sterni <sternenseemann@systemli.org> Reviewed-by: tazjin <tazjin@tvl.su>
Diffstat (limited to 'tvix/eval/src/opcode.rs')
| -rw-r--r-- | tvix/eval/src/opcode.rs | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tvix/eval/src/opcode.rs b/tvix/eval/src/opcode.rs index 130e242668d6..e3eddc3c9ad0 100644 --- a/tvix/eval/src/opcode.rs +++ b/tvix/eval/src/opcode.rs @@ -129,6 +129,7 @@ pub enum OpCode { // Type assertion operators OpAssertBool, + OpAssertAttrs, /// Access local identifiers with statically known positions. OpGetLocal(StackIdx), |