RISC-V: Remove implicit enablement of Zvknha from Zvkn.

Message ID 20240621015800.13808-1-wangfeng@eswincomputing.com
State New
Headers
Series RISC-V: Remove implicit enablement of Zvknha from Zvkn. |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_binutils_build--master-arm warning Patch is already merged
linaro-tcwg-bot/tcwg_binutils_build--master-aarch64 warning Patch is already merged

Commit Message

Feng Wang June 21, 2024, 1:58 a.m. UTC
  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

Nelson Chu June 21, 2024, 5:03 a.m. UTC | #1
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
>
>
  

Patch

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},