From: plugd <plugd@thelambdalab.xyz>
Date: Thu, 14 Nov 2019 13:37:33 +0000 (+0100)
Subject: Fixed up address arithmetic.
X-Git-Url: https://thelambdalab.xyz/gitweb/index.cgi?p=jars.git;a=commitdiff_plain;h=871e290a730e6f7a2e6c7dd8ffb754f2007d5f74;hp=707bcb205c185815e2b94ab3fca5f769afb58264

Fixed up address arithmetic.
---

diff --git a/mars.scm b/mars.scm
index b1aa9db..f2e3d99 100644
--- a/mars.scm
+++ b/mars.scm
@@ -17,7 +17,10 @@
 ;;
 
 (define core (make-vector core-size '()))
-
+(define (addr+ . args)
+  (foldl (lambda (a b)
+           (modulo (+ a b core-size) core-size))
+         0 args))
 
 ;;; Instructions
 ;;
@@ -68,9 +71,10 @@
 (define (execute-instr ptr)
   (let* ((instr (vector-ref core ptr))
          (A-pointer (eval-operand (instr-A-mode instr) (instr-A-num) ptr))
-         (B-pointer (eval-operand (instr-B-mode instr) (instr-B-num) ptr)))
+         (B-pointer (eval-operand (instr-B-mode instr) (instr-B-num) ptr))
+         (modifier (instr-modifier instr)))
     (case (instr-opcode instr)
-      ((DAT))
+      ((DAT) '()) ;Game over, man, game over!
       ((MOV))
       ((ADD))
       ((SUB))
@@ -92,25 +96,25 @@
   (case mode
     ((immediate) 0)
     ((direct) num)
-    ((indirect-A) (+ num (instr-A-num (vector-ref core (+ ptr num)))))
-    ((indirect-B) (+ num (instr-B-num (vector-ref core (+ ptr num)))))
+    ((indirect-A) (addr+ num (instr-A-num (vector-ref core (addr+ ptr num)))))
+    ((indirect-B) (addr+ num (instr-B-num (vector-ref core (addr+ ptr num)))))
     ((pre-indirect-A)
-     (let ((aux-instr (vector-ref core (+ ptr num))))
-       (instr-set-A-num! aux-instr (- 1 (instr-A-num aux-instr)))
-       (+ num (instr-A-num aux-instr))))
+     (let ((aux-instr (vector-ref core (addr+ ptr num))))
+       (instr-set-A-num! aux-instr (addr+ -1 (instr-A-num aux-instr)))
+       (addr+ num (instr-A-num aux-instr))))
     ((pre-indirect-B)
-     (let ((aux-instr (vector-ref core (+ ptr num))))
-       (instr-set-B-num! aux-instr (- 1 (instr-B-num aux-instr)))
-       (+ num (instr-B-num aux-instr))))
+     (let ((aux-instr (vector-ref core (addr+ ptr num))))
+       (instr-set-B-num! aux-instr (addr+ -1 (instr-B-num aux-instr)))
+       (addr+ num (instr-B-num aux-instr))))
     ((post-indirect-A)
-     (let* ((aux-instr (vector-ref core (+ ptr num)))
+     (let* ((aux-instr (vector-ref core (addr+ ptr num)))
             (old-A-num (instr-A-num aux-instr)))
-       (instr-set-A-num! aux-instr (+ 1 (instr-A-num aux-instr)))
-       (+ num old-A-num)))
+       (instr-set-A-num! aux-instr (addr+ 1 (instr-A-num aux-instr)))
+       (addr+ num old-A-num)))
     ((post-indirect-B)
-     (let* ((aux-instr (vector-ref core (+ ptr num)))
+     (let* ((aux-instr (vector-ref core (addr+ ptr num)))
             (old-B-num (instr-B-num aux-instr)))
-       (instr-set-B-num! aux-instr (+ 1 (instr-B-num aux-instr)))
-       (+ num old-B-num)))
+       (instr-set-B-num! aux-instr (addr+ 1 (instr-B-num aux-instr)))
+       (addr+ num old-B-num)))
     (else
      (error "Unrecognized mode" mode))))