9745: Addressing review feedback

barretenberg/cpp/pil/avm/main.pil

@@ -514,7 +514,7 @@ namespace main(256);
     //===== Memory Slice Constraints ============================================
     pol commit sel_slice_gadget; // Selector to activate a slice gadget operation in the gadget (#[PERM_MAIN_SLICE]).
 
-    // Activate only if tag_err is disabled or retsize (ib) is non-zero
+    // Activate only if op_err is disabled or retsize (ib) is non-zero
     ib * (1 - op_err) * (sel_op_calldata_copy + sel_op_external_return - sel_slice_gadget)= 0;
 
     //====== Inter-table Constraints ============================================

barretenberg/cpp/src/barretenberg/vm/avm/trace/trace.cpp

@@ -1438,6 +1438,7 @@ AvmError AvmTraceBuilder::op_get_env_var(uint8_t indirect, uint8_t env_var, uint
             op_dagasleft(indirect, dst_offset);
             break;
         default:
+            // Cannot happen thanks to the first if clause. This is to make the compiler happy.
             throw std::runtime_error("Invalid environment variable");
             break;
         }
@@ -2548,7 +2549,6 @@ AvmError AvmTraceBuilder::op_nullifier_exists(uint8_t indirect,
 {
     auto const clk = static_cast<uint32_t>(main_trace.size()) + 1;
 
-    // TODO: resolved_address should be used
     auto [resolved_nullifier_offset, resolved_address, resolved_dest] =
         Addressing<3>::fromWire(indirect, call_ptr)
             .resolve({ nullifier_offset, address_offset, dest_offset }, mem_trace_builder);