RISC-V: Imply zicsr for sdtrig and ssstrict extensions.
Checks
| Context |
Check |
Description |
| rivoscibot/toolchain-ci-rivos-lint |
warning
|
Lint failed
|
| rivoscibot/toolchain-ci-rivos-apply-patch |
success
|
Patch applied
|
| rivoscibot/toolchain-ci-rivos-build--newlib-rv64gcv-lp64d-multilib |
success
|
Build passed
|
| rivoscibot/toolchain-ci-rivos-build--linux-rv64gc_zba_zbb_zbc_zbs-lp64d-multilib |
success
|
Build passed
|
| rivoscibot/toolchain-ci-rivos-build--linux-rv64gcv-lp64d-multilib |
success
|
Build passed
|
| rivoscibot/toolchain-ci-rivos-test |
success
|
Testing passed
|
Commit Message
This patch implies zicsr for sdtrig and ssstrict extensions.
According to the riscv-privileged spec, the sdtrig and ssstrict extensions
are privileged extensions, so they should imply zicsr.
gcc/ChangeLog:
* config/riscv/riscv-ext.def: Imply zicsr.
---
gcc/config/riscv/riscv-ext.def | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--
2.43.0
Comments
On 9/12/25 20:17, Dongyan Chen wrote:
> This patch implies zicsr for sdtrig and ssstrict extensions.
> According to the riscv-privileged spec, the sdtrig and ssstrict extensions
> are privileged extensions, so they should imply zicsr.
>
> gcc/ChangeLog:
>
> * config/riscv/riscv-ext.def: Imply zicsr.
Thanks. I've pushed this to the trunk.
jeff
@@ -1564,7 +1564,7 @@ DEFINE_RISCV_EXT(
/* FULL_NAME */ "sdtrig extension",
/* DESC */ "",
/* URL */ ,
- /* DEP_EXTS */ ({}),
+ /* DEP_EXTS */ ({"zicsr"}),
/* SUPPORTED_VERSIONS */ ({{1, 0}}),
/* FLAG_GROUP */ sd,
/* BITMASK_GROUP_ID */ BITMASK_NOT_YET_ALLOCATED,
@@ -1941,7 +1941,7 @@ DEFINE_RISCV_EXT(
/* FULL_NAME */ "ssstrict extension",
/* DESC */ "",
/* URL */ ,
- /* DEP_EXTS */ ({}),
+ /* DEP_EXTS */ ({"zicsr"}),
/* SUPPORTED_VERSIONS */ ({{1, 0}}),
/* FLAG_GROUP */ ss,
/* BITMASK_GROUP_ID */ BITMASK_NOT_YET_ALLOCATED,