Merge pull request #7 from daniellehrner/feat/stack-refactor

Replace heap-allocated Stack with fixed-size array

Author: garyschulte

build.zig

@@ -328,6 +328,20 @@ pub fn build(b: *std.Build) void {
     const test_step = b.step("test", "Run unit tests");
     test_step.dependOn(&run_tests.step);
 
+    // Inline zig tests for interpreter module (discovers tests in all imported files)
+    const interpreter_tests = b.addTest(.{
+        .root_module = b.createModule(.{
+            .root_source_file = .{ .src_path = .{ .owner = b, .sub_path = "src/interpreter/main.zig" } },
+            .target = target,
+            .optimize = optimize,
+        }),
+    });
+    interpreter_tests.root_module.addImport("primitives", primitives_module);
+    interpreter_tests.root_module.addImport("bytecode", bytecode_module);
+    interpreter_tests.root_module.addImport("context", context_module);
+    const run_interpreter_tests = b.addRunArtifact(interpreter_tests);
+    test_step.dependOn(&run_interpreter_tests.step);
+
     // Precompile unit tests - these are run via zig test command in CI
     // The command needs to link libc and include all modules
     // See .github/workflows/ci.yml for the full command

examples/benchmark.zig

@@ -68,7 +68,6 @@ fn benchmarkStackOperations() !void {
 
     const iterations = 100_000;
     var stack = interpreter.Stack.new();
-    defer stack.deinit();
 
     const start_time = std.time.nanoTimestamp();
 

src/handler/main.zig

@@ -145,7 +145,6 @@ pub const FrameResult = struct {
     pub fn deinit(self: *FrameResult) void {
         self.result.deinit();
         self.memory.deinit();
-        self.stack.deinit();
     }
 };
 

src/interpreter/interpreter.zig

@@ -391,7 +391,6 @@ pub const Interpreter = struct {
     /// Deinitialize the interpreter
     pub fn deinit(self: *Interpreter) void {
         self.bytecode.deinit();
-        self.stack.deinit();
         self.memory.deinit();
     }
 

src/interpreter/main.zig

@@ -45,19 +45,6 @@ pub const testing = struct {
         std.debug.print("Gas tests passed.\n", .{});
     }
 
-    pub fn testStack() !void {
-        var stack = Stack.new();
-        std.debug.assert(stack.len() == 0);
-
-        try stack.push(@as(primitives.U256, 1));
-        std.debug.assert(stack.len() == 1);
-
-        const value = stack.pop() orelse return error.StackEmpty;
-        std.debug.assert(value == @as(primitives.U256, 1));
-
-        std.debug.print("Stack tests passed.\n", .{});
-    }
-
     pub fn testMemory() !void {
         var memory = Memory.new();
         std.debug.assert(memory.size() == 0);