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

What is the current status of the standard? #831

Closed
ChunyuLiao opened this issue Jul 10, 2023 · 7 comments
Closed

What is the current status of the standard? #831

ChunyuLiao opened this issue Jul 10, 2023 · 7 comments
Assignees
Labels
Type:Question For general questions

Comments

@ChunyuLiao
Copy link

If not frozen, is there a plan for when it will be frozen?

We are submitting patch to upstream llvm and need to know the current status of the standard.

@MikeOpenHWGroup
Copy link
Member

Hi @ChunyuLiao, thanks for your interest in CV32E40P. What "standard" are you asking about? The XPULP ISA? Probably the best person to comment on this is @pascalgouedo, but he is on personal leave for a couple of weeks. @jeremybennett can you comment?

@ChunyuLiao
Copy link
Author

ChunyuLiao commented Jul 12, 2023

thanks @MikeOpenHWGroup
I'm asking about CV32E40P. the last release is cv32e40p_v1.3.2 . I'm not sure what XPULP ISA is.

As a standard extension. there are five Specification States.
our CV32E40P, as a vendor extension, is there a plan for when it will be frozen.

@MikeOpenHWGroup
Copy link
Member

Please note that the XPULP ISA is not a RISC-V standard ISA and does not follow the "five states" of a RISC-V International specification. Rather, it is a custom vendor extension that is explicitly tied to the implementation of the CV32E40P. So we can consider the XPULP ISA to be frozen when release v2.0.0 of the CV32E40P is made. I believe the goal for this is October of this year.

I am aware that there has been some discussion about creating a more formal specification (beyond what is published in the CV32E40P User Manual), but do not know if these discussions are moving forward. If you are interested in seeing this happen, you could consider joining the OpenHW Group and making a proposal.

@ChunyuLiao
Copy link
Author

Thanks, looking forward to the CV32E40P freezing soon.

@MikeOpenHWGroup MikeOpenHWGroup added the Type:Question For general questions label Jul 13, 2023
@MikeOpenHWGroup
Copy link
Member

Hi @ChunyuLiao, if I have answered your question, please close this issue. Thanks!

@jeremybennett
Copy link
Contributor

jeremybennett commented Jul 14, 2023

@MikeOpenHWGroup I think this answered the wrong question from @ChunyuLiao . This is the specification of the core. From a compiler perspective it is the numbering of the versions of the ISA which matters for upstream acceptance. This was frozen earlier this year (see corev-isa-extension-naming.md).

The tool chain is not supporting versioning of the ISA extensions (which is going to become the standard practice for RISC-V builtins). The only ISA extensions we shall support are those added to CV32E40Pv2. Since there is only one version of the ISA extensions supported in the tool chains, they must be (per RISC-V standards) ISA extension version 1.0.0.

I'll submit a patch to the documentation of the ISA extensions in the main document to make it clear that the ISA extension version from a compiler perspective is 1.0.0. This has caused a disproportionate amount of review feedback in the upstream Clang/LLVM submission.

@jeremybennett jeremybennett reopened this Jul 14, 2023
@jeremybennett
Copy link
Contributor

Closing again after adding the supplementary comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Question For general questions
Projects
None yet
Development

No branches or pull requests

3 participants