RISC-V: Remove implicit enablement of Zvknha from Zvkn.
Checks
Commit Message
Accroding to the Crypto spec, the Zvkned,Zvknhb,Zvkb and Zvkt are
included in the Zvkn. So the Zvknha should be removed from Zvkn.
bfd/ChangeLog:
* elfxx-riscv.c: Remove zvknha from zvkn.
---
bfd/elfxx-riscv.c | 1 -
1 file changed, 1 deletion(-)
Comments
https://github.com/riscv/riscv-crypto/blob/v20230620/doc/vector/riscv-crypto-vector-zvkn.adoc
Yeah, according to the above spec link, that makes sense, so committed,
thanks.
Nelson
On Fri, Jun 21, 2024 at 9:58 AM Feng Wang <wangfeng@eswincomputing.com>
wrote:
> Accroding to the Crypto spec, the Zvkned,Zvknhb,Zvkb and Zvkt are
> included in the Zvkn. So the Zvknha should be removed from Zvkn.
>
> bfd/ChangeLog:
>
> * elfxx-riscv.c: Remove zvknha from zvkn.
> ---
> bfd/elfxx-riscv.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index 297d565285c..fbd0e7ba0e1 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1252,7 +1252,6 @@ static struct riscv_implicit_subset
> riscv_implicit_subsets[] =
> {"zks", "zksh", check_implicit_always},
> {"zvbb", "zvkb", check_implicit_always},
> {"zvkn", "zvkned", check_implicit_always},
> - {"zvkn", "zvknha", check_implicit_always},
> {"zvkn", "zvknhb", check_implicit_always},
> {"zvkn", "zvkb", check_implicit_always},
> {"zvkn", "zvkt", check_implicit_always},
> --
> 2.17.1
>
>
@@ -1252,7 +1252,6 @@ static struct riscv_implicit_subset riscv_implicit_subsets[] =
{"zks", "zksh", check_implicit_always},
{"zvbb", "zvkb", check_implicit_always},
{"zvkn", "zvkned", check_implicit_always},
- {"zvkn", "zvknha", check_implicit_always},
{"zvkn", "zvknhb", check_implicit_always},
{"zvkn", "zvkb", check_implicit_always},
{"zvkn", "zvkt", check_implicit_always},