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

Remove outdated RVV 2.0 notice from document #1402 #1406

Closed
wants to merge 1 commit into from
Closed

Remove outdated RVV 2.0 notice from document #1402 #1406

wants to merge 1 commit into from

Conversation

camel-cdr
Copy link

For reasons I describe here, I think it's best to remove the RVV 2.0 version notice from the specification, there does seem to be no indication that what the notice says is actually planned.

I replaced it with text derived from the Vector Crypto Introduction, both should ideally be reworded before an actual release, but this should remove the confusion for now.

See also past discussion on the v-spec repo: riscvarchive/riscv-v-spec#946

It sounded like the change would be merged, but the repo was archived before that got through. My understanding is that the riscv-v-spec repo wasn't supposed to be a source of truth, hence it was archived.

@aswaterman
Copy link
Member

I'm dropping the extensive warning about what Ratified means, as it's a global property of RVIA policy, not of this extension. Yes, I understand there's a tortured history in this case, but I'd rather not flog a dead horse.

@aswaterman
Copy link
Member

aswaterman commented May 14, 2024

Looking at the other extensions (with the exception of vector crypto), the consistent thing to do is to remove any draft/freeze warnings and update the preface. That's what we'll do here.

57ec29a

@aswaterman aswaterman closed this May 14, 2024
@camel-cdr
Copy link
Author

I'm dropping the extensive warning about what Ratified means, as it's a global property of RVIA policy, not of this extension

Exactly, that was the motivation behind this change. Everything I've heard from RVI in any official capacity other than this NOTE referred to RVV 1.0 being ratified. Maybe I've missed some part of the communication.

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