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

Create capcache64 test, change capcache to RV32 #42

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

elliotb-lowrisc
Copy link
Contributor

The capcache test was using RV64 instructions without requiring has_xlen_64.
Create a new test called capcache64 that checks for and uses the RV64 instructions.
Change capcache to only use RV32 instructions by changing generators from gen_rv64_Xcheri_cache to gen_rv32_Xcheri_cache. This follows the same pattern as the float and float64 tests.

The capcache test was using RV64 instructions without
requiring `has_xlen_64`.
Create a new test called capcache64 that checks for and uses
the RV64 instructions.
Change capcache to only use RV32 instructions by changing generators
from `gen_rv64_Xcheri_cache` to `gen_rv32_Xcheri_cache`.
This follows the same pattern as the float and float64 tests.
@PeterRugg
Copy link
Collaborator

This is probably a good idea, but for what it's worth I'm currently working on a big overhaul for how architecture requirements get specified and used!

@elliotb-lowrisc
Copy link
Contributor Author

Oh? Sounds good. Shall I close this then?

@PeterRugg
Copy link
Collaborator

Feel free to go ahead if this fixes your problem, although I don't see the new rv32 test anywhere? Did it already exist?

@elliotb-lowrisc
Copy link
Contributor Author

Yep, it already existed. Just needed using it seemed.

@PeterRugg PeterRugg merged commit a646ed2 into CTSRD-CHERI:master Apr 22, 2024
2 checks passed
@elliotb-lowrisc elliotb-lowrisc deleted the capcache branch May 3, 2024 13:09
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