openhwgroup / corev-binutils-gdb

GNU General Public License v2.0
9 stars 26 forks source link

[CV32E40Pv2] Compiler error for cv.shuffleI*.sci.* #67

Closed pascalgouedo closed 1 year ago

pascalgouedo commented 1 year ago

For all cv.shuffleI.sci. instructions, the specification is maybe not clear enough but immediate values should always be treated as unsigned. Because this immediate value is not representing a number (its sub-parts are individually used as indexes).

Specification will be improved for that point.

NandniJamnadas commented 1 year ago

There is a pull request for this fix: CV32E40Pv2 SIMD 6-bit Unsigned Immediate + Updated SIMD instructions from signed to unsigned, @pascalgouedo could you please check if the correct instructions listed use a 6-bit unsigned immediate.

jeremybennett commented 1 year ago

This should be resolved by https://github.com/openhwgroup/corev-binutils-gdb/pull/74.