OpenCores
Issue List
Typo in m8080.v and sm8080.v #4
Open NULL opened this issue about 2 years ago
NULL commented about 2 years ago

These versions of the design have a typo in the STAX instruction:

if (opcode4) // use DE pair waddrhold <= regfil`reg_d<<8|regfil`reg_d;

reg_d is used twice.

NULL commented about 2 years ago

Same problem in LDAX a few lines down.

NULL commented about 2 years ago

Another typo in m8080.v:

              // the illegal opcodes behave as NOPs

              6'b001011, 6'b011001, 6'b011101, 6'b101101, 
              6'b111101: begin

                 state <= `cpus_fetchi; // fetch next instruction
                 pc <= pc + 2'b10; // Next instruction byte, cns 2

              end
								The PC is incremented wrong.

Assignee
No one
Labels
Bug