Skip to content

Added RISCV v6 release notes#2846

Merged
Rot127 merged 3 commits intocapstone-engine:nextfrom
moste00:docs/riscv_in_v6_relase_notes
Feb 1, 2026
Merged

Added RISCV v6 release notes#2846
Rot127 merged 3 commits intocapstone-engine:nextfrom
moste00:docs/riscv_in_v6_relase_notes

Conversation

@moste00
Copy link
Contributor

@moste00 moste00 commented Jan 10, 2026

Your checklist for this pull request

  • I've documented or updated the documentation of every API function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

...

Test plan

...

Closing issues

...

@moste00
Copy link
Contributor Author

moste00 commented Jan 10, 2026

#2756

Copy link

@notxvilka notxvilka left a comment

Choose a reason for hiding this comment

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

@wargio take a look too please

@moste00 moste00 force-pushed the docs/riscv_in_v6_relase_notes branch from c49e4b3 to 79236cc Compare January 13, 2026 01:09
@moste00
Copy link
Contributor Author

moste00 commented Jan 18, 2026

@Rot127 would you please see if the current status are good and approve is so so that we can merge ?

Copy link
Collaborator

@Rot127 Rot127 left a comment

Choose a reason for hiding this comment

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

Just those small things. Mind though that I will merge this after the main RISC-V PR was merged.

* `CS_MODE_RISCV_ZBKC = 1 << 18`
* `CS_MODE_RISCV_ZBKX = 1 << 19`
* `CS_MODE_RISCV_ZBS = 1 << 20`
* NOTE1: All extensions above are disabled by default unless enabled by their option name or the corresponding command line flag in cstool. Any other extension is always enabled and can't be disabled.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* NOTE1: All extensions above are disabled by default unless enabled by their option name or the corresponding command line flag in cstool. Any other extension is always enabled and can't be disabled.
* NOTE: All extensions above are disabled by default unless enabled by their option name or the corresponding command line flag in cstool. Any other extension is always enabled and can't be disabled.

* `CS_MODE_RISCV_ZBS = 1 << 20`
* NOTE1: All extensions above are disabled by default unless enabled by their option name or the corresponding command line flag in cstool. Any other extension is always enabled and can't be disabled.

* NOTE2: RISC-V has a massive, sprawling list of extensions, but Capstone's internal implementaton choice of using a 32-bit mode field is not enough to cover all of them. For now, those extension flags above were added because their encoding space is conflicting with either each other or other extensions. More flags can be added later if bug reports come in requesting finer-grained extension control. However, the current implementation using bitfields imposes a strict upper limit and would likely be refactored for a more expansive mechanism in the future. See [this issue](https://github.com/capstone-engine/capstone/issues/2848) for more details.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* NOTE2: RISC-V has a massive, sprawling list of extensions, but Capstone's internal implementaton choice of using a 32-bit mode field is not enough to cover all of them. For now, those extension flags above were added because their encoding space is conflicting with either each other or other extensions. More flags can be added later if bug reports come in requesting finer-grained extension control. However, the current implementation using bitfields imposes a strict upper limit and would likely be refactored for a more expansive mechanism in the future. See [this issue](https://github.com/capstone-engine/capstone/issues/2848) for more details.
* NOTE: RISC-V has a massive, sprawling list of extensions, but Capstone's internal implementaton choice of using a 32-bit mode field is not enough to cover all of them. For now, those extension flags above were added because their encoding space is conflicting with either each other or other extensions. More flags can be added later if bug reports come in requesting finer-grained extension control. However, the current implementation using bitfields imposes a strict upper limit and would likely be refactored for a more expansive mechanism in the future. See [this issue](https://github.com/capstone-engine/capstone/issues/2848) for more details.

Copy link
Collaborator

Choose a reason for hiding this comment

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

You can also consider using the Github markdown for note highlighting. Looks a little more fancy: https://github.com/orgs/community/discussions/16925

@moste00 moste00 force-pushed the docs/riscv_in_v6_relase_notes branch from 3652cee to cb14a28 Compare January 21, 2026 21:37
@moste00 moste00 force-pushed the docs/riscv_in_v6_relase_notes branch from cb14a28 to 59320f7 Compare January 21, 2026 21:38
@Rot127
Copy link
Collaborator

Rot127 commented Jan 21, 2026

Sorry, the note didn't show because of the indentation. Now I messed up the formatting even more :D

@moste00
Copy link
Contributor Author

moste00 commented Jan 21, 2026

Thank you <3 for a while I thought the issue magically fixed itself because I was looking at the rich diff and wondering why it was broken then I saw the "Refresh" button and voila it was fixed !

I think we're done here and we will return to this when we merge Capstone main and the LLVM fork.

@Rot127
Copy link
Collaborator

Rot127 commented Jan 21, 2026

it was broken then I saw the "Refresh" button and voila it was fixed !

Lol, the rich diff makes it look super weird. I thought it is still broken. But looks good if the file is normally showed, all good now.

@Rot127 Rot127 merged commit d2d73fe into capstone-engine:next Feb 1, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants