kuopinghsu / srv32

Simple 3-stage pipeline RISC-V processor
MIT License
133 stars 25 forks source link

fix-Bash syntax error in riscv-compliace test #4

Closed WeiCheng14159 closed 3 years ago

WeiCheng14159 commented 3 years ago

The bash shell used to switch between riscv-compliace test version 1/2 is problematic. When test_v variable is set to 1, the master branch of riscv-compliance is cloned instead of branch v1.0. This is a common bash shell syntax error as explained in this stackoverflow thread. (https://stackoverflow.com/questions/2011160/unexpected-operator-error)

WeiCheng14159 commented 3 years ago

After this fix, 48/48 rv32i, 8/8 rv32im, 6/6 rv32Zicsr riscv compliance (ver. 1) should pass without mistake.