Technically there are two valid NOPs

2022-12-24 01:34:16 -07:00 · 2022-12-24 01:34:16 -07:00 · 8ef06a7e1a
commit 8ef06a7e1a
parent 51d6cdd3b1
1 changed files with 2 additions and 2 deletions
--- a/src/vm.rs
+++ b/src/vm.rs
@ -139,8 +139,8 @@ impl Uxn {
                // The value of PC is defined to be the value of the NEXT pc ala Mips
                self.pc += 1;

-                // Short circuit for fast NOP
-                if icode == Icode::NOP {
+                // Short circuit for cheap NOPs (POPk, POPkr)
+                if icode & Icode::NOP == Icode::NOP {
                    return Ok(());
                }