alaasal/GPCore

instr_decoder

Opened this issue · 1 comments

I noticed the following is unused code. If the lower 2 bits of any opcode != 2'b11 then the instruction is a Compressed Instruction and should cause a trap if you don't support Compressed Instructions or instructions larger than 32 bits

assign noOp = ~(|op); // if op code = 000000 then set opcode0 to 1

Hello

Thanks for the note. I totally understand what you are saying.

Currently, the core needs a lot of cleaning but we are in a two week break now so hopefully by mid-March it will be totally cleaned. And also there is another branch where csrs and traps are being implemented but not yet merged and/or finished.