196. Pla implícito

registrar PLA implícito.

Lección 196 de 356 · tests/chapter_01_cpu/test_196_PLA_implied.py

Requisito previo: el paso 195 añadió `pla. En este paso, modifica únicamente emulator/cpu/opcodes.py importando pla y añadiendo su entrada en OPCODE_TABLE`.

Por qué existe este paso

La entrada de PLA es la pila de hardware seleccionada por S. El opcode $68 no tiene operando, así que `CPU.step() debe despachar directamente a pla(cpu)`.

Implementación sugerida

from emulator.cpu.instructions import pla

OPCODE_TABLE = {
    # existing entries
    0x68: pla,
}

Invariantes: conserva las asignaciones existentes; asigna exactamente $68 a `pla`; no uses envoltorio de modo de direccionamiento; consume solo el byte de opcode; conserva los efectos sobre Z/N del paso 195.

Concepto erróneo: la memoria de programa tras $68 no proporciona el valor extraído; lo hace la dirección de pila ya incrementada.

Fuera de alcance: el comportamiento de `pla` es el paso 195. PHP empieza en el paso 197, y esta transición no debe introducir su función ni su asignación de opcode.

Ejecutar esta lección

uv run pytest tests/chapter_01_cpu/test_196_PLA_implied.py -v

Implementación de ejemplo

Implementación sugerida

from emulator.cpu.instructions import pla

OPCODE_TABLE = {
    # existing entries
    0x68: pla,
}

Ver código del testtest_196_PLA_implied.py

import inspect

from emulator.bus.cpu_bus import CpuBus
from emulator.cpu import opcodes
from emulator.cpu.cpu import CPU
from emulator.cpu.instructions import pla
from emulator.memory.fake_rom import FakeROM


STACK_BASE = 0x0100


def make_cpu_with_rom():
    rom = FakeROM()
    rom.write(0x7FFC, 0x00)
    rom.write(0x7FFD, 0x80)
    bus = CpuBus(program_rom=rom)
    return CPU(bus), bus, rom


def test_pla_implied_is_in_opcode_table():
    """Objective: opcode 0x68 is the official PLA opcode."""
    assert opcodes.OPCODE_TABLE[0x68] is pla


def test_pla_instruction_signature_takes_only_cpu():
    """Objective: PLA is implied, so pla(cpu) does not need an operand argument."""
    assert list(inspect.signature(pla).parameters) == ["cpu"]


def test_opcode_68_pla_pulls_stack_value_into_accumulator():
    """Objective: executing opcode 0x68 pulls the next stack byte into A."""
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x68)

    cpu.reset()
    cpu.s = 0xFC
    bus.write(STACK_BASE | 0xFD, 0x42)
    cpu.step()

    assert cpu.a == 0x42
    assert cpu.s == 0xFD


def test_opcode_68_pla_updates_zero_and_negative_flags():
    """Objective: PLA opcode behavior includes Zero/Negative flag updates."""
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x68)

    cpu.reset()
    cpu.s = 0xFC
    bus.write(STACK_BASE | 0xFD, 0x80)
    cpu.step()

    assert cpu.a == 0x80
    assert cpu.flags.get_zero_flag() is False
    assert cpu.flags.get_negative_flag() is True


def test_opcode_68_pla_does_not_fetch_operand_bytes():
    """
    Objective:
    PLA is one byte long. The byte after PLA must not be consumed as an operand.

    The pulled value comes from the stack, not from program memory after 0x68.
    """
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x68)
    rom.write(0x0001, 0x99)

    cpu.reset()
    cpu.s = 0xFC
    bus.write(STACK_BASE | 0xFD, 0x55)
    cpu.step()

    assert cpu.a == 0x55
    assert cpu.pc == 0x8001


def test_opcode_68_pla_preserves_other_flags():
    """Objective: PLA updates Z/N but does not damage unrelated status flags."""
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x68)

    cpu.reset()
    cpu.s = 0xFC
    cpu.p = 0b0100_0001
    bus.write(STACK_BASE | 0xFD, 0x01)
    cpu.step()

    assert cpu.a == 0x01
    assert cpu.p & 0b0100_0001 == 0b0100_0001
    assert cpu.flags.get_zero_flag() is False
    assert cpu.flags.get_negative_flag() is False

Funciones de test (6)

test_pla_implied_is_in_opcode_table
test_pla_instruction_signature_takes_only_cpu
test_opcode_68_pla_pulls_stack_value_into_accumulator
test_opcode_68_pla_updates_zero_and_negative_flags
test_opcode_68_pla_does_not_fetch_operand_bytes
test_opcode_68_pla_preserves_other_flags