From 1b3f1f5b1f230732f2d7deea3e206a53a4e984d4 Mon Sep 17 00:00:00 2001 From: Bradford Lamson-Scribner Date: Sun, 31 May 2020 22:21:04 -0600 Subject: [PATCH] internal/vm/exec_funcs: remove the todo exec func! --- internal/vm/exec_funcs.go | 61 +++++++++++++++++---------------------- internal/vm/vm.go | 4 +-- 2 files changed, 29 insertions(+), 36 deletions(-) diff --git a/internal/vm/exec_funcs.go b/internal/vm/exec_funcs.go index 217938d..d395e50 100644 --- a/internal/vm/exec_funcs.go +++ b/internal/vm/exec_funcs.go @@ -1,12 +1,5 @@ package vm -import "fmt" - -func todo(a *Appleone, o op) error { - fmt.Println("implement me") - return nil -} - // interrupt, N Z C I D V // push PC+2, push SR - - - 1 - - func execBRK(a *Appleone, o op) error { @@ -41,7 +34,7 @@ func execDEC(a *Appleone, o op) error { b-- a.mem[addr] = b a.maybeSetFlagZero(b) - a.maybeHandleOverflow(b) + a.maybeSetFlagOverflow(b) return nil } @@ -56,7 +49,7 @@ func execINC(a *Appleone, o op) error { b++ a.mem[addr] = b a.maybeSetFlagZero(b) - a.maybeHandleOverflow(b) + a.maybeSetFlagOverflow(b) return nil } @@ -65,7 +58,7 @@ func execINC(a *Appleone, o op) error { func execINX(a *Appleone, o op) error { a.cpu.x++ a.maybeSetFlagZero(a.cpu.x) - a.maybeHandleOverflow(a.cpu.x) + a.maybeSetFlagOverflow(a.cpu.x) return nil } @@ -74,7 +67,7 @@ func execINX(a *Appleone, o op) error { func execINY(a *Appleone, o op) error { a.cpu.y++ a.maybeSetFlagZero(a.cpu.y) - a.maybeHandleOverflow(a.cpu.y) + a.maybeSetFlagOverflow(a.cpu.y) return nil } @@ -83,7 +76,7 @@ func execINY(a *Appleone, o op) error { func execTAX(a *Appleone, o op) error { a.cpu.x = a.cpu.a a.maybeSetFlagZero(a.cpu.x) - a.maybeHandleOverflow(a.cpu.x) + a.maybeSetFlagOverflow(a.cpu.x) return nil } @@ -92,7 +85,7 @@ func execTAX(a *Appleone, o op) error { func execTAY(a *Appleone, o op) error { a.cpu.y = a.cpu.a a.maybeSetFlagZero(a.cpu.y) - a.maybeHandleOverflow(a.cpu.y) + a.maybeSetFlagOverflow(a.cpu.y) return nil } @@ -101,7 +94,7 @@ func execTAY(a *Appleone, o op) error { func execDEX(a *Appleone, o op) error { a.cpu.x-- a.maybeSetFlagZero(a.cpu.x) - a.maybeHandleOverflow(a.cpu.x) + a.maybeSetFlagOverflow(a.cpu.x) return nil } @@ -110,7 +103,7 @@ func execDEX(a *Appleone, o op) error { func execDEY(a *Appleone, o op) error { a.cpu.y-- a.maybeSetFlagZero(a.cpu.y) - a.maybeHandleOverflow(a.cpu.y) + a.maybeSetFlagOverflow(a.cpu.y) return nil } @@ -123,7 +116,7 @@ func execLDA(a *Appleone, o op) error { } a.cpu.a = operand a.maybeSetFlagZero(a.cpu.a) - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) return nil } @@ -136,7 +129,7 @@ func execLDX(a *Appleone, o op) error { } a.cpu.x = operand a.maybeSetFlagZero(a.cpu.x) - a.maybeHandleOverflow(a.cpu.x) + a.maybeSetFlagOverflow(a.cpu.x) return nil } @@ -149,7 +142,7 @@ func execLDY(a *Appleone, o op) error { } a.cpu.y = operand a.maybeSetFlagZero(a.cpu.y) - a.maybeHandleOverflow(a.cpu.y) + a.maybeSetFlagOverflow(a.cpu.y) return nil } @@ -177,7 +170,7 @@ func execADC(a *Appleone, o op) error { } a.maybeSetFlagZero(a.cpu.a) - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) return nil } @@ -212,7 +205,7 @@ func execSBC(a *Appleone, o op) error { } a.maybeSetFlagZero(a.cpu.a) - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) return nil } @@ -299,7 +292,7 @@ func execBIT(a *Appleone, o op) error { a.setFlag(flagOverflow) } - a.maybeHandleOverflow(operand) + a.maybeSetFlagOverflow(operand) return nil } @@ -358,7 +351,7 @@ func execEOR(a *Appleone, o op) error { return err } a.cpu.a ^= operand - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) a.maybeSetFlagZero(a.cpu.a) return nil } @@ -470,7 +463,7 @@ func execPHA(a *Appleone, o op) error { // + + - - - - func execTXA(a *Appleone, o op) error { a.cpu.a = a.cpu.x - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) a.maybeSetFlagZero(a.cpu.a) return nil } @@ -479,7 +472,7 @@ func execTXA(a *Appleone, o op) error { // + + - - - - func execTYA(a *Appleone, o op) error { a.cpu.a = a.cpu.y - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) a.maybeSetFlagZero(a.cpu.a) return nil } @@ -488,7 +481,7 @@ func execTYA(a *Appleone, o op) error { // + + - - - - func execTSX(a *Appleone, o op) error { a.cpu.x = a.cpu.sp - a.maybeHandleOverflow(a.cpu.x) + a.maybeSetFlagOverflow(a.cpu.x) a.maybeSetFlagZero(a.cpu.x) return nil } @@ -497,7 +490,7 @@ func execTSX(a *Appleone, o op) error { // + + - - - - func execPLA(a *Appleone, o op) error { a.cpu.a = a.popStackWord() - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) a.maybeSetFlagZero(a.cpu.a) return nil } @@ -505,7 +498,7 @@ func execPLA(a *Appleone, o op) error { // pull SR from stack N Z C I D V func execPLP(a *Appleone, o op) error { a.cpu.ps = a.popStackWord() | 0B_00110000 - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) a.maybeSetFlagZero(a.cpu.a) return nil } @@ -555,7 +548,7 @@ func execLSR(a *Appleone, o op) error { } a.maybeSetFlagZero(operand) - a.maybeHandleOverflow(operand) + a.maybeSetFlagOverflow(operand) if o.addrMode == accumulator { a.cpu.a = operand @@ -597,7 +590,7 @@ func execROL(a *Appleone, o op) error { } a.maybeSetFlagZero(operand) - a.maybeHandleOverflow(operand) + a.maybeSetFlagOverflow(operand) if o.addrMode == accumulator { a.cpu.a = operand @@ -619,7 +612,7 @@ func execTXS(a *Appleone, o op) error { a.cpu.sp = a.cpu.x // TODO: needed? // a.maybeSetFlagZero(a.cpu.sp) - // a.maybeHandleOverflow(a.cpu.sp) + // a.maybeSetFlagOverflow(a.cpu.sp) return nil } @@ -649,7 +642,7 @@ func execROR(a *Appleone, o op) error { } a.maybeSetFlagZero(operand) - a.maybeHandleOverflow(operand) + a.maybeSetFlagOverflow(operand) if o.addrMode == accumulator { a.cpu.a = operand @@ -682,7 +675,7 @@ func execASL(a *Appleone, o op) error { operand <<= 1 a.maybeSetFlagZero(operand) - a.maybeHandleOverflow(operand) + a.maybeSetFlagOverflow(operand) if o.addrMode == accumulator { a.cpu.a = operand @@ -707,7 +700,7 @@ func execAND(a *Appleone, o op) error { } a.cpu.a &= operand a.maybeSetFlagZero(a.cpu.a) - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) return nil } @@ -720,6 +713,6 @@ func execORA(a *Appleone, o op) error { } a.cpu.a |= operand a.maybeSetFlagZero(a.cpu.a) - a.maybeHandleOverflow(a.cpu.a) + a.maybeSetFlagOverflow(a.cpu.a) return nil } diff --git a/internal/vm/vm.go b/internal/vm/vm.go index 5eaa672..0d993a9 100644 --- a/internal/vm/vm.go +++ b/internal/vm/vm.go @@ -147,7 +147,7 @@ func (a *Appleone) clearFlag(flag byte) { a.cpu.ps &^= flag } -func (a *Appleone) maybeHandleOverflow(word byte) { +func (a *Appleone) maybeSetFlagOverflow(word byte) { a.clearFlag(flagNegative) if word > 127 { a.setFlag(flagNegative) @@ -185,5 +185,5 @@ func (a *Appleone) compare(b1, b2 byte) { } b := byte(uint16(b1) - uint16(b2)) - a.maybeHandleOverflow(b) + a.maybeSetFlagOverflow(b) }