Skip to content

Conversation

a4lg
Copy link

@a4lg a4lg commented Aug 30, 2025

This PR directly corresponds to rust-lang/rust#145948 (and this is a superset of #2002).

@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Aug 30, 2025
@a4lg a4lg force-pushed the riscv-extensions-2025.1 branch 2 times, most recently from c60bc59 to 25d43f0 Compare August 30, 2025 03:17
@a4lg a4lg force-pushed the riscv-extensions-2025.1 branch from 25d43f0 to a35f1c8 Compare August 30, 2025 04:18
@a4lg
Copy link
Author

a4lg commented Aug 30, 2025

Possible Consideration

Some platform behavior extensions are hard to make a short, concise summary.

Do we prefer to keep long(er than usual) descriptions? Or, do we just link to the RISC-V Profiles documentation and let it explain?

My Current Position

I prefer the latter if allowed but mdbook design is confirmed not broken so the current approach (long descriptions) is okay for me.

@ehuss ehuss added the S-waiting-on-stabilization Waiting for a stabilization PR to be merged in the main Rust repository label Sep 2, 2025
@a4lg
Copy link
Author

a4lg commented Sep 11, 2025

Ping. Can someone review this PR?

@a4lg
Copy link
Author

a4lg commented Sep 17, 2025

Non-stabilization part is separated as #2002.

Submission v2 (zacas is added) of this PR is a superset of #2002.

@a4lg a4lg force-pushed the riscv-extensions-2025.1 branch from a35f1c8 to d27acfa Compare September 17, 2025 12:45
@a4lg a4lg changed the title RISC-V Extensions update including 28 extensions to stabilize RISC-V Extensions update including 29 extensions to stabilize Sep 17, 2025
The `Zb` extension does not exist and we instead have the `B` extension
which is a superset of the three subextensions: `Zba`, `Zbb` and `Zbs`.

For now (since `B` extension is not stabilized on Rust),
this commit just removes `Zb`.
This is the latest ratified ISA Manual version and more importantly,
some individual documents are merged into this.

So, this commit consistently use this Manual (this version) where possible.

Along with this, this commit substitutes link IDs from `rv-zb-*` to `rv-*`
because `rv-zb-*` will look like the different document (bit-manipulation)
is referenced.
rust-lang/rust#140139 changed the target feature implication without
updating the Reference.  That change is came from the fact that the Zbc
extension is a functional superset of the Zbkc extension.

This commit fixes this issue.

Appendix - List of all Zbkc / Zbc instructions:

*   `clmul`  (Zbkc / Zbc)
*   `clmulh` (Zbkc / Zbc)
*   `clmulr` (Zbc only)
This commit directly corresponds to rust-lang/rust#145948.

References:

1.  RISC-V Instruction Set Manual (version 20250508):
    <https://github.com/riscv/riscv-isa-manual/tree/20250508>
2.  RISC-V Profiles (version 1.0 - RVA23 is not stabilized at the time):
    <https://github.com/riscv/riscv-profiles/tree/v1.0>
3.  RISC-V Profiles (RVA23/RVB23-ratified version):
    <https://github.com/riscv/riscv-profiles/tree/rva23-rvb23-ratified>
@a4lg a4lg force-pushed the riscv-extensions-2025.1 branch from d27acfa to 5527d55 Compare September 18, 2025 01:02
@a4lg
Copy link
Author

a4lg commented Sep 18, 2025

cc: @Amanieu, @RalfJung

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.

4 participants