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

[CV32E40P] Update the pulp instruction tests to use the encoding supported by the latest embecosm sdk #2547

Open
Guilherme-Soares-Sequeira opened this issue Oct 29, 2024 · 1 comment

Comments

@Guilherme-Soares-Sequeira

Description

With the release of CV32E40Pv2 all of the instructions were reencoded. All of the instruction prefixes were changed to "cv", and some instructions were hardcoded with .word directives which no longer work as expected with the latest sdk. When creating the CV32E40Pv2 tag, this should be fixed.

Task Outcome

The tests should be refactored / fixed to take into account the reencoding of instructions.

Location Information

The tests are located in "cv32e40p/tests/programs/custom/pulp*"

Completion Criteria

The tests are able to be compiled and ran with the newest SDK.

Additional context

This task was alluded to or mentioned in #2520 and here.
If it is of use, I adapted the tests so that they are runnable within X-HEEP. While there were a lot of changes which are not required or desirable for this project, it may be of some use. Also, I wrote some tests for hardware loops, which aren't present in this repository. It is available here.
I also wrote some documentation on how the instruction prefixes changed and overall problems faced when adapting the tests, which may also be of use. It is available here and here.

@MikeOpenHWGroup
Copy link
Member

Hi @Guilherme-Soares-Sequeira, thanks for your interest in the CV32E40Pv2. You are probably not aware that the verification environment for the v2 is still on the cv32e40p/dev branch of this repo. The plan is to make that branch available on master when we formally release v2.0.0 of the CV32E40P.

Please have a look at test-programs on the cv32e40p/dev branch.

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

2 participants