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

[dv] Psuedo instruction coverage for Bitmanip #1926

Open
GregAC opened this issue Nov 13, 2022 · 0 comments
Open

[dv] Psuedo instruction coverage for Bitmanip #1926

GregAC opened this issue Nov 13, 2022 · 0 comments
Milestone

Comments

@GregAC
Copy link
Contributor

GregAC commented Nov 13, 2022

Three bitmanip instructions seem to be missing coverage, orc.b, zext.h and rev8. They are being executed but they can be interpreted as pseudo-instruction forms of more general instructions or instructions in their own right depending on what bitmanip implementation you are using.

We should consider reworking our coverage collection to ensure executing these samples both the pseudo instruction form and the general instruction form (so it works with configs that don't have the general instruction form).

In the case of zext.h we don't implement the more general pack it uses but have to emit it in that form from the Ibex tracer due to LLVM/GCC tooling issues, see #1228

@GregAC GregAC added this to the V3 milestone Nov 13, 2022
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