riscv-isa-sim icon indicating copy to clipboard operation
riscv-isa-sim copied to clipboard

Support more than 65 Z* extensions

Open aswaterman opened this issue 2 years ago • 0 comments

The isa_extension_t enum already has 44 extensions. In not too long, the enum will grow in size to 65, when it will collide with the 'A' extension. Fix that preemptively by starting after 'Z'.

This approach will run out of steam at 165 extensions because we are using unsigned char to represent extensions, but opefully we will have retired by that point.

In seriousness, we will probably need to refactor the extension_enabled logic at some point in the future (e.g. when the configuration structure is finally added) and at that point we should lift the char limit.

aswaterman avatar Nov 18 '22 00:11 aswaterman