philpax / skiron

Skiron is a basic 32-bit RISC-inspired CPU architecture.
MIT License
3 stars 1 forks source link

Fix 16-bit offset issues #21

Closed philpax closed 8 years ago

philpax commented 8 years ago

loadui, loadli, and SplitAbsolute relocation is broken as a result of the opcode changes. A new encoding needs to be introduced to allow for 16-bit immediates. Could be accomplished by increasing Encoding bitfield size by 1, and dropping Variant and OperandSize from the resulting encoding.

philpax commented 8 years ago

Fixed by rearranging the fields and making OperandSize optional for Encoding B such that there are 14-bit/16-bit immediates, depending on the opcode.