Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TVM explenation in readme needs update? #473

Open
defermelowie opened this issue May 24, 2023 · 0 comments
Open

TVM explenation in readme needs update? #473

defermelowie opened this issue May 24, 2023 · 0 comments

Comments

@defermelowie
Copy link

defermelowie commented May 24, 2023

In the current README.md, I found the following table of all existing TVMs. However, when checking the described riscv_test.h, I found a different set. This set is listed in the second table below. Is it correct that the table in README.md is outdated or did I misinterpret it and is there no 1-to-1 mapping with the macros in riscv_test.h?

TVM Name Description
rv32ui RV32 user-level, integer only
rv32si RV32 supervisor-level, integer only
rv64ui RV64 user-level, integer only
rv64uf RV64 user-level, integer and floating-point
rv64uv RV64 user-level, integer, floating-point, and vector
rv64si RV64 supervisor-level, integer only
rv64sv RV64 supervisor-level, integer and vector
TVM macro name Supposed description
RVTEST_RV32M machine-level, RV32I
RVTEST_RV32S supervisor-level, RV32I
RVTEST_RV32U user-level, RV32I
RVTEST_RV32UF user-level, RV32IF
RVTEST_RV32UV user-level, RV32IFV
RVTEST_RV64M machine-level, RV64I
RVTEST_RV64S supervisor-level, RV64I
RVTEST_RV64U user-level, RV64I
RVTEST_RV64UF user-level, RV64IF
RVTEST_RV64UV user-level, RV64IFV
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant