From da9e885e06664c2dcd0e76034495970591640ded Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Sat, 17 Sep 2022 19:10:40 +0300 Subject: refactor(tvix/eval): rename OpAttrsIsSet -> OpHasAttr This matches the name of the AST node from which it was compiled. Suggested by sterni in cl/6231 Change-Id: Ia51525158d2f47467c01fce2282005b1a8417a47 Reviewed-on: https://cl.tvl.fyi/c/depot/+/6623 Tested-by: BuildkiteCI Reviewed-by: sterni Reviewed-by: grfn --- tvix/eval/src/compiler/attrs.rs | 2 +- tvix/eval/src/opcode.rs | 2 +- tvix/eval/src/vm.rs | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tvix/eval/src/compiler/attrs.rs b/tvix/eval/src/compiler/attrs.rs index e2e932953b7c..f1441cff68d2 100644 --- a/tvix/eval/src/compiler/attrs.rs +++ b/tvix/eval/src/compiler/attrs.rs @@ -256,7 +256,7 @@ impl Compiler<'_, '_> { // After the last fragment, emit the actual instruction that // leaves a boolean on the stack. - self.push_op(OpCode::OpAttrsIsSet, &node); + self.push_op(OpCode::OpHasAttr, &node); } pub(super) fn compile_select(&mut self, slot: LocalIdx, node: ast::Select) { diff --git a/tvix/eval/src/opcode.rs b/tvix/eval/src/opcode.rs index 706aceac8357..9d7fba9ee453 100644 --- a/tvix/eval/src/opcode.rs +++ b/tvix/eval/src/opcode.rs @@ -91,7 +91,7 @@ pub enum OpCode { OpAttrsUpdate, OpAttrsSelect, OpAttrsTrySelect, - OpAttrsIsSet, + OpHasAttr, // `with`-handling OpPushWith(StackIdx), diff --git a/tvix/eval/src/vm.rs b/tvix/eval/src/vm.rs index 980a3ed0f5dd..a0eb7d8ad2ee 100644 --- a/tvix/eval/src/vm.rs +++ b/tvix/eval/src/vm.rs @@ -317,7 +317,7 @@ impl<'o> VM<'o> { self.push(value); } - OpCode::OpAttrsIsSet => { + OpCode::OpHasAttr => { let key = fallible!(self, self.pop().to_str()); let result = match self.pop() { Value::Attrs(attrs) => attrs.contains(key.as_str()), -- cgit 1.4.1