search

ZipCPU / zipcpu

A small, light weight, RISC CPU soft core
1.27k stars 153 forks source link

wrong comment #19

Closed IvanVnucec closed 2 years ago

IvanVnucec commented 2 years ago

Here below:
https://github.com/ZipCPU/zipcpu/blob/548a0efbe5729279f2ae9779a2af4229e8241e52/sw/gas-zippatch.patch#L169

ZipCPU commented 2 years ago

Thank you. Yes, it does look like that comment is a bit out of date. The ZipCPU was originally a 32-bits per byte machine. Back when each address referenced 32-bits, a 32-bit address essentially accessed 2^34 octets of data. It now only has 8-bits per byte, so the comment "There's 32 bits per byte" in the line following is also out of date.

Dan

IvanVnucec commented 2 years ago

thank you for an explanation. god bless

ZipCPU commented 2 years ago

This has now been fixed in #43a0cdb.

Dan