xc.prot specified with cshamt field rather than crs2

scarv / xcrypto

XCrypto: a cryptographic ISE for RISC-V

MIT License

92 stars 10 forks source link

xc.prot specified with cshamt field rather than crs2 #67

Closed ben-marshall closed 5 years ago

ben-marshall commented 5 years ago

The xc.prot instruction should be a packed shift of one register value by the corresponding packed elements in another register value. Insteat, the crs2 register field it should use is actually a cshamt field.

How this wasn't spotted I do not know.

Blocks #57

ben-marshall commented 5 years ago

Problem not present in binutils patch, looks like it is just a spec problem.