Merge pull request #1426 from o1-labs/feature/mips/and

Implement `and`

optimism/src/mips/interpreter.rs

@@ -597,6 +597,19 @@ pub trait InterpreterEnv {
         position: Self::Position,
     ) -> Self::Variable;
 
+    /// Returns `x or y`, storing the result in `position`.
+    ///
+    /// # Safety
+    ///
+    /// There are no constraints on the returned value; callers must manually add constraints to
+    /// ensure that it is correctly constructed.
+    unsafe fn and_witness(
+        &mut self,
+        x: &Self::Variable,
+        y: &Self::Variable,
+        position: Self::Position,
+    ) -> Self::Variable;
+
     /// Returns `x or y`, storing the result in `position`.
     ///
     /// # Safety
@@ -792,7 +805,19 @@ pub fn interpret_rtype<Env: InterpreterEnv>(env: &mut Env, instr: RTypeInstructi
         }
         RTypeInstruction::Sub => (),
         RTypeInstruction::SubUnsigned => (),
-        RTypeInstruction::And => (),
+        RTypeInstruction::And => {
+            let rs = env.read_register(&rs);
+            let rt = env.read_register(&rt);
+            let res = {
+                // FIXME: Constrain
+                let pos = env.alloc_scratch();
+                unsafe { env.and_witness(&rs, &rt, pos) }
+            };
+            env.write_register(&rd, res);
+            env.set_instruction_pointer(next_instruction_pointer.clone());
+            env.set_next_instruction_pointer(next_instruction_pointer + Env::constant(4u32));
+            return;
+        }
         RTypeInstruction::Or => {
             let rs = env.read_register(&rs);
             let rt = env.read_register(&rt);

optimism/src/mips/witness.rs

@@ -277,6 +277,17 @@ impl<Fp: Field> InterpreterEnv for Env<Fp> {
         res
     }
 
+    unsafe fn and_witness(
+        &mut self,
+        x: &Self::Variable,
+        y: &Self::Variable,
+        position: Self::Position,
+    ) -> Self::Variable {
+        let res = x & y;
+        self.write_column(position, res.into());
+        res
+    }
+
     unsafe fn or_witness(
         &mut self,
         x: &Self::Variable,