Skip to content

Document RISC-V d, e, and f extensions#2274

Open
ehuss wants to merge 1 commit into
rust-lang:masterfrom
ehuss:risc-def
Open

Document RISC-V d, e, and f extensions#2274
ehuss wants to merge 1 commit into
rust-lang:masterfrom
ehuss:risc-def

Conversation

@ehuss
Copy link
Copy Markdown
Contributor

@ehuss ehuss commented May 24, 2026

These are being stabilized in rust-lang/rust#156188 as cfg-only features. It is a little odd to list them here in the target_feature attribute section since they can't be used with the attribute, but I think it makes sense to keep things together. Perhaps in the future we may consider breaking these lists out into a separate section and rearrange things.

These are being stabilized in
rust-lang/rust#156188 as cfg-only features. It
is a little odd to list them here in the target_feature attribute
section since they can't be used with the attribute, but I think it
makes sense to keep things together. Perhaps in the future we may
consider breaking these lists out into a separate section and rearrange
things.
@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label May 24, 2026
@ehuss
Copy link
Copy Markdown
Contributor Author

ehuss commented May 24, 2026

@romancardenas can you review and confirm everything looks right here? In particular, I was uncertain about the E feature, since it has a slightly different way of being documented in the manual.

Also, I noticed that we are linking to the 20250508 tag, which is a bit old now. Would it make sense to change the links to a newer version? I've noticed these features have had some updates. I think the latest ratified version is v20260120.

@ehuss ehuss added the S-waiting-on-stabilization Waiting for a stabilization PR to be merged in the main Rust repository label May 24, 2026
Copy link
Copy Markdown

@romancardenas romancardenas left a comment

Choose a reason for hiding this comment

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

LGTM! Regarding updating the links to the RISC-V specification, I think It should be OK. At least these extensions have not experienced too many changes.

View changes since this review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: The marked PR is awaiting review from a maintainer S-waiting-on-stabilization Waiting for a stabilization PR to be merged in the main Rust repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants