Skip to content

RISC-V exhaustive CSR tests#257

Open
moste00 wants to merge 2 commits intorizinorg:masterfrom
moste00:riscv_csr_test
Open

RISC-V exhaustive CSR tests#257
moste00 wants to merge 2 commits intorizinorg:masterfrom
moste00:riscv_csr_test

Conversation

@moste00
Copy link
Copy Markdown
Contributor

@moste00 moste00 commented Feb 20, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@notxvilka notxvilka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this for?

@moste00
Copy link
Copy Markdown
Contributor Author

moste00 commented Mar 1, 2026

What is this for?

I wanted to add to rizin DWARF register numbers for CSR registers and map them to string names, so to test this I wanted to make a binary containing a bunch of them, it's certainly a bit excessive in scope (AI-written) but I like to be as extensive as possible when doing test coverage.

The reason it's not too urgent/frozen is because I later found that DWARF info almost never references CSRs usually, so the corresponding code in the rizin repo is not published in a PR yet until I make sure it realistically gets used in some real world case even if it's very niche, but I will likely not find such a case and delete the code so it doesn't become a burden.

In all cases I think the test binary is useful as test cases for CSR instructions coverage intendent of anything.

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

Successfully merging this pull request may close these issues.

2 participants