Skip to content
New issue

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

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

Already on GitHub? # to your account

Updated wfe test to work correctly without user-mode/PMP, added flags… #2255

Merged

Conversation

silabs-hfegran
Copy link
Contributor

@silabs-hfegran silabs-hfegran commented Oct 23, 2023

… to fix ISS coverage issue

  • Fixed: Wfe test did not support running without PMP/User mode
  • Added: --excludem RISCV_CSR_UNIMP flag as per imperas' instructions to avoid errors when sampling coverage for illegal CSR instructions (w/ non-implemented CSRs)

… to fix ISS coverage issue

Signed-off-by: Henrik Fegran <Henrik.Fegran@silabs.com>
Signed-off-by: Henrik Fegran <Henrik.Fegran@silabs.com>
Copy link
Contributor

@silabs-robin silabs-robin left a comment

Choose a reason for hiding this comment

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

Approving based on:
It looks like this will make the test passable on the X.

@silabs-robin silabs-robin merged commit 7c2a400 into openhwgroup:cv32e40s/dev Oct 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants