[PING,3/3] RISC-V: Add support for Zvfbfwma extension

Message ID 20240605013613.78830-4-zengxiao@eswincomputing.com
State New
Headers
Series RISC-V: Add support for Zfbfmin Zvfbfmin and Zvfbfwma extension |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_binutils_build--master-arm success Testing passed
linaro-tcwg-bot/tcwg_binutils_build--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_binutils_check--master-aarch64 success Testing passed
linaro-tcwg-bot/tcwg_binutils_check--master-arm success Testing passed

Commit Message

Xiao Zeng June 5, 2024, 1:36 a.m. UTC
  This implements the Zvfbfwma extension, as of version 1.0.
View detailed information in:
<https://github.com/riscv/riscv-isa-manual/blob/main/src/bfloat16.adoc#zvfbfwma---vector-bf16-widening-mul-add>

1 In spec: "Zvfbfwma requires the Zvfbfmin extension and the Zfbfmin extension."
  1.1 In Embedded    Processor: Zvfbfwma -> Zvfbfmin -> Zve32f
  1.2 In Application Processor: Zvfbfwma -> Zvfbfmin -> V
  1.3 In both scenarios, there are: Zvfbfwma -> Zfbfmin

2 Depending on different usage scenarios, the Zvfbfwma extension may
depend on 'V' or 'Zve32f'. This patch only implements dependencies in
scenario of Embedded Processor. This is consistent with the processing
strategy in Zvfbfmin. In scenario of Application Processor, it is
necessary to explicitly indicate the dependent 'V' extension.

For relevant information in gcc, please refer to:
<https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=38dd4e26e07c6be7cf4d169141ee4f3a03f3a09d>

bfd/ChangeLog:

	* elfxx-riscv.c (riscv_multi_subset_supports): Handle Zvfbfwma.
	(riscv_multi_subset_supports_ext): Ditto.

gas/ChangeLog:

	* NEWS: Updated.
	* testsuite/gas/riscv/march-help.l: Ditto.
	* testsuite/gas/riscv/zvfbfwma-rv32.d: New test.
	* testsuite/gas/riscv/zvfbfwma-rv32.s: New test.
	* testsuite/gas/riscv/zvfbfwma-rv64.d: New test.
	* testsuite/gas/riscv/zvfbfwma-rv64.s: New test.

include/ChangeLog:

	* opcode/riscv-opc.h (MATCH_VFWMACCBF16_VF): Define.
	(MASK_VFWMACCBF16_VF): Ditto.
	(MATCH_VFWMACCBF16_VV): Ditto.
	(MASK_VFWMACCBF16_VV): Ditto.
	(DECLARE_INSN): New declarations for Zvfbfwma.
	* opcode/riscv.h (enum riscv_insn_class): Add
	INSN_CLASS_ZVFBFWMA

opcodes/ChangeLog:

	* riscv-opc.c: Add Zvfbfwma instructions.
---
 bfd/elfxx-riscv.c                       |  7 +++++++
 gas/NEWS                                |  2 ++
 gas/testsuite/gas/riscv/march-help.l    |  1 +
 gas/testsuite/gas/riscv/zvfbfwma-rv32.d | 12 ++++++++++++
 gas/testsuite/gas/riscv/zvfbfwma-rv32.s |  7 +++++++
 gas/testsuite/gas/riscv/zvfbfwma-rv64.d | 12 ++++++++++++
 gas/testsuite/gas/riscv/zvfbfwma-rv64.s |  7 +++++++
 include/opcode/riscv-opc.h              |  8 ++++++++
 include/opcode/riscv.h                  |  1 +
 opcodes/riscv-opc.c                     |  4 ++++
 10 files changed, 61 insertions(+)
 create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv32.d
 create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv32.s
 create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv64.d
 create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv64.s
  

Comments

Nelson Chu June 6, 2024, 2:45 a.m. UTC | #1
On Wed, Jun 5, 2024 at 9:30 AM Xiao Zeng <zengxiao@eswincomputing.com>
wrote:

> This implements the Zvfbfwma extension, as of version 1.0.
> View detailed information in:
> <
> https://github.com/riscv/riscv-isa-manual/blob/main/src/bfloat16.adoc#zvfbfwma---vector-bf16-widening-mul-add
> >
>
> 1 In spec: "Zvfbfwma requires the Zvfbfmin extension and the Zfbfmin
> extension."
>   1.1 In Embedded    Processor: Zvfbfwma -> Zvfbfmin -> Zve32f
>   1.2 In Application Processor: Zvfbfwma -> Zvfbfmin -> V
>   1.3 In both scenarios, there are: Zvfbfwma -> Zfbfmin
>
> 2 Depending on different usage scenarios, the Zvfbfwma extension may
> depend on 'V' or 'Zve32f'. This patch only implements dependencies in
> scenario of Embedded Processor. This is consistent with the processing
> strategy in Zvfbfmin. In scenario of Application Processor, it is
> necessary to explicitly indicate the dependent 'V' extension.
>
> For relevant information in gcc, please refer to:
> <
> https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=38dd4e26e07c6be7cf4d169141ee4f3a03f3a09d
> >
>
> bfd/ChangeLog:
>
>         * elfxx-riscv.c (riscv_multi_subset_supports): Handle Zvfbfwma.
>         (riscv_multi_subset_supports_ext): Ditto.
>
> gas/ChangeLog:
>
>         * NEWS: Updated.
>         * testsuite/gas/riscv/march-help.l: Ditto.
>         * testsuite/gas/riscv/zvfbfwma-rv32.d: New test.
>         * testsuite/gas/riscv/zvfbfwma-rv32.s: New test.
>         * testsuite/gas/riscv/zvfbfwma-rv64.d: New test.
>         * testsuite/gas/riscv/zvfbfwma-rv64.s: New test.
>
> include/ChangeLog:
>
>         * opcode/riscv-opc.h (MATCH_VFWMACCBF16_VF): Define.
>         (MASK_VFWMACCBF16_VF): Ditto.
>         (MATCH_VFWMACCBF16_VV): Ditto.
>         (MASK_VFWMACCBF16_VV): Ditto.
>         (DECLARE_INSN): New declarations for Zvfbfwma.
>         * opcode/riscv.h (enum riscv_insn_class): Add
>         INSN_CLASS_ZVFBFWMA
>
> opcodes/ChangeLog:
>
>         * riscv-opc.c: Add Zvfbfwma instructions.
> ---
>  bfd/elfxx-riscv.c                       |  7 +++++++
>  gas/NEWS                                |  2 ++
>  gas/testsuite/gas/riscv/march-help.l    |  1 +
>  gas/testsuite/gas/riscv/zvfbfwma-rv32.d | 12 ++++++++++++
>  gas/testsuite/gas/riscv/zvfbfwma-rv32.s |  7 +++++++
>  gas/testsuite/gas/riscv/zvfbfwma-rv64.d | 12 ++++++++++++
>  gas/testsuite/gas/riscv/zvfbfwma-rv64.s |  7 +++++++
>  include/opcode/riscv-opc.h              |  8 ++++++++
>  include/opcode/riscv.h                  |  1 +
>  opcodes/riscv-opc.c                     |  4 ++++
>  10 files changed, 61 insertions(+)
>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv32.d
>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv32.s
>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv64.d
>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv64.s
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index 3d303f02b58..1182c25cab2 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1193,6 +1193,8 @@ static struct riscv_implicit_subset
> riscv_implicit_subsets[] =
>    {"v", "zvl128b",     check_implicit_always},
>    {"zabha", "a",       check_implicit_always},
>    {"zvfbfmin", "zve32f",       check_implicit_always},
> +  {"zvfbfwma", "zve32f",       check_implicit_always},
> +  {"zvfbfwma", "zfbfmin",      check_implicit_always},
>    {"zvfh", "zvfhmin",  check_implicit_always},
>    {"zvfh", "zfhmin",   check_implicit_always},
>    {"zvfhmin", "zve32f",        check_implicit_always},
> @@ -1396,6 +1398,7 @@ static struct riscv_supported_ext
> riscv_supported_std_z_ext[] =
>    {"zvbc",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>    {"zvfh",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>    {"zvfbfmin",         ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
> +  {"zvfbfwma",         ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>    {"zvfhmin",          ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>    {"zvkb",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>    {"zvkg",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
> @@ -2648,6 +2651,8 @@ riscv_multi_subset_supports (riscv_parse_subset_t
> *rps,
>        return riscv_subset_supports (rps, "zvbc");
>      case INSN_CLASS_ZVFBFMIN:
>        return riscv_subset_supports (rps, "zvfbfmin");
> +    case INSN_CLASS_ZVFBFWMA:
> +      return riscv_subset_supports (rps, "zvfbfwma");
>      case INSN_CLASS_ZVKB:
>        return riscv_subset_supports (rps, "zvkb");
>      case INSN_CLASS_ZVKG:
> @@ -2914,6 +2919,8 @@ riscv_multi_subset_supports_ext
> (riscv_parse_subset_t *rps,
>        return _("zvbc");
>      case INSN_CLASS_ZVFBFMIN:
>        return "zvfbfmin";
> +    case INSN_CLASS_ZVFBFWMA:
> +      return "zvfbfwma";
>      case INSN_CLASS_ZVKB:
>        return _("zvkb");
>      case INSN_CLASS_ZVKG:
> diff --git a/gas/NEWS b/gas/NEWS
> index 2c75966d0ce..45eafeff38f 100644
> --- a/gas/NEWS
> +++ b/gas/NEWS
> @@ -1,5 +1,7 @@
>  -*- text -*-
>
> +* Add support for RISC-V Zvfbfwma extension with version 1.0.
> +
>  * Add support for RISC-V Zvfbfmin extension with version 1.0.
>
>  * Add support for RISC-V Zfbfmin extension with version 1.0.
> diff --git a/gas/testsuite/gas/riscv/march-help.l
> b/gas/testsuite/gas/riscv/march-help.l
> index 1a2ac1eaa08..4b051b189b4 100644
> --- a/gas/testsuite/gas/riscv/march-help.l
> +++ b/gas/testsuite/gas/riscv/march-help.l
> @@ -60,6 +60,7 @@ All available -march extensions for RISC-V:
>         zvbc                                    1.0
>         zvfh                                    1.0
>         zvfbfmin                                1.0
> +       zvfbfwma                                1.0
>         zvfhmin                                 1.0
>         zvkb                                    1.0
>         zvkg                                    1.0
> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
> b/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
> new file mode 100644
> index 00000000000..2c00dabc32a
> --- /dev/null
> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
> @@ -0,0 +1,12 @@
> +#as: -march=rv32i_zvfbfwma
> +#objdump: -d
> +
> +.*:[   ]+file format .*
> +
> +Disassembly of section .text:
> +
> +0+000 <target>:
> +[      ]+[0-9a-f]+:[   ]+ee865257[     ]+vfwmaccbf16.vf[       ]+v4,fa2,v8
> +[      ]+[0-9a-f]+:[   ]+ec865257[     ]+vfwmaccbf16.vf[
>  ]+v4,fa2,v8,v0.t
> +[      ]+[0-9a-f]+:[   ]+ee861257[     ]+vfwmaccbf16.vv[       ]+v4,v12,v8
> +[      ]+[0-9a-f]+:[   ]+ec861257[     ]+vfwmaccbf16.vv[
>  ]+v4,v12,v8,v0.t
> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
> b/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
> new file mode 100644
> index 00000000000..f824af98361
> --- /dev/null
> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
> @@ -0,0 +1,7 @@
> +target:
> +       # vfwmaccbf16.vf
> +       vfwmaccbf16.vf v4, fa2, v8
> +       vfwmaccbf16.vf v4, fa2, v8, v0.t
> +       # vfwmaccbf16.vv
> +       vfwmaccbf16.vv v4, v12, v8
> +       vfwmaccbf16.vv v4, v12, v8, v0.t
>

No need to seperate rv32 and rv64 test cases.


> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
> b/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
> new file mode 100644
> index 00000000000..05da1328eea
> --- /dev/null
> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
> @@ -0,0 +1,12 @@
> +#as: -march=rv64iv_zvfbfwma
> +#objdump: -d
> +
> +.*:[   ]+file format .*
> +
> +Disassembly of section .text:
> +
> +0+000 <target>:
> +[      ]+[0-9a-f]+:[   ]+ee865257[     ]+vfwmaccbf16.vf[       ]+v4,fa2,v8
> +[      ]+[0-9a-f]+:[   ]+ec865257[     ]+vfwmaccbf16.vf[
>  ]+v4,fa2,v8,v0.t
> +[      ]+[0-9a-f]+:[   ]+ee861257[     ]+vfwmaccbf16.vv[       ]+v4,v12,v8
> +[      ]+[0-9a-f]+:[   ]+ec861257[     ]+vfwmaccbf16.vv[
>  ]+v4,v12,v8,v0.t
> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
> b/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
> new file mode 100644
> index 00000000000..f824af98361
> --- /dev/null
> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
> @@ -0,0 +1,7 @@
> +target:
> +       # vfwmaccbf16.vf
> +       vfwmaccbf16.vf v4, fa2, v8
> +       vfwmaccbf16.vf v4, fa2, v8, v0.t
> +       # vfwmaccbf16.vv
> +       vfwmaccbf16.vv v4, v12, v8
> +       vfwmaccbf16.vv v4, v12, v8, v0.t
> diff --git a/include/opcode/riscv-opc.h b/include/opcode/riscv-opc.h
> index 32b971fb2b3..7db3dd1a1ed 100644
> --- a/include/opcode/riscv-opc.h
> +++ b/include/opcode/riscv-opc.h
> @@ -2375,6 +2375,11 @@
>  #define MASK_VFNCVTBF16_F_F_W 0xfc0ff07f
>  #define MATCH_VFWCVTBF16_F_F_V 0x48069057
>  #define MASK_VFWCVTBF16_F_F_V 0xfc0ff07f
> +/* Zvfbfwma intructions.  */
> +#define MATCH_VFWMACCBF16_VF 0xec005057
> +#define MASK_VFWMACCBF16_VF 0xfc00707f
> +#define MATCH_VFWMACCBF16_VV 0xec001057
> +#define MASK_VFWMACCBF16_VV 0xfc00707f
>  /* Vendor-specific (CORE-V) Xcvmac instructions.  */
>  #define MATCH_CV_MAC       0x9000302b
>  #define MASK_CV_MAC        0xfe00707f
> @@ -3928,6 +3933,9 @@ DECLARE_INSN(FCVT_S_BF16, MATCH_FCVT_S_BF16,
> MASK_FCVT_S_BF16)
>  /* Zvfbfmin instructions.  */
>  DECLARE_INSN(VFNCVTBF16_F_F_W, MATCH_VFNCVTBF16_F_F_W,
> MASK_VFNCVTBF16_F_F_W)
>  DECLARE_INSN(VFWCVTBF16_F_F_V, MATCH_VFWCVTBF16_F_F_V,
> MASK_VFWCVTBF16_F_F_V)
> +/* Zvfbfwma instructions.  */
> +DECLARE_INSN(VFWMACCBF16_VF, MATCH_VFWMACCBF16_VF, MASK_VFWMACCBF16_VF)
> +DECLARE_INSN(VFWMACCBF16_VV, MATCH_VFWMACCBF16_VV, MASK_VFWMACCBF16_VV)
>  /* Zvbb/Zvkb instructions.  */
>  DECLARE_INSN(vandn_vv, MATCH_VANDN_VV, MASK_VANDN_VV)
>  DECLARE_INSN(vandn_vx, MATCH_VANDN_VX, MASK_VANDN_VX)
> diff --git a/include/opcode/riscv.h b/include/opcode/riscv.h
> index 4d21b6c3926..1ebfc5210ca 100644
> --- a/include/opcode/riscv.h
> +++ b/include/opcode/riscv.h
> @@ -472,6 +472,7 @@ enum riscv_insn_class
>    INSN_CLASS_ZVBB,
>    INSN_CLASS_ZVBC,
>    INSN_CLASS_ZVFBFMIN,
> +  INSN_CLASS_ZVFBFWMA,
>    INSN_CLASS_ZVKB,
>    INSN_CLASS_ZVKG,
>    INSN_CLASS_ZVKNED,
> diff --git a/opcodes/riscv-opc.c b/opcodes/riscv-opc.c
> index 0a470aee7cc..670e986978c 100644
> --- a/opcodes/riscv-opc.c
> +++ b/opcodes/riscv-opc.c
> @@ -2045,6 +2045,10 @@ const struct riscv_opcode riscv_opcodes[] =
>  {"vfncvtbf16.f.f.w", 0, INSN_CLASS_ZVFBFMIN, "Vd,VtVm",
> MATCH_VFNCVTBF16_F_F_W, MASK_VFNCVTBF16_F_F_W, match_opcode, 0},
>  {"vfwcvtbf16.f.f.v", 0, INSN_CLASS_ZVFBFMIN, "Vd,VtVm",
> MATCH_VFWCVTBF16_F_F_V, MASK_VFWCVTBF16_F_F_V, match_opcode, 0},
>
> +/* Zvfbfwma instructions.  */
> +{"vfwmaccbf16.vf",  0, INSN_CLASS_ZVFBFWMA, "Vd,S,VtVm",
> MATCH_VFWMACCBF16_VF, MASK_VFWMACCBF16_VF, match_opcode, 0},
> +{"vfwmaccbf16.vv",  0, INSN_CLASS_ZVFBFWMA, "Vd,Vs,VtVm",
> MATCH_VFWMACCBF16_VV, MASK_VFWMACCBF16_VV, match_opcode, 0},
> +
>  /* Zvkg instructions.  */
>  {"vghsh.vv",   0, INSN_CLASS_ZVKG, "Vd,Vt,Vs", MATCH_VGHSH_VV,
> MASK_VGHSH_VV, match_opcode, 0},
>  {"vgmul.vv",   0, INSN_CLASS_ZVKG, "Vd,Vt", MATCH_VGMUL_VV,
> MASK_VGMUL_VV, match_opcode, 0},
> --
> 2.17.1
>
>
  
Xiao Zeng June 6, 2024, 6:24 a.m. UTC | #2
2024-06-06 10:45  Nelson Chu <nelson@rivosinc.com> wrote:
>
>On Wed, Jun 5, 2024 at 9:30 AM Xiao Zeng <zengxiao@eswincomputing.com>
>wrote:
>
>> This implements the Zvfbfwma extension, as of version 1.0.
>> View detailed information in:
>> <
>> https://github.com/riscv/riscv-isa-manual/blob/main/src/bfloat16.adoc#zvfbfwma---vector-bf16-widening-mul-add
>> >
>>
>> 1 In spec: "Zvfbfwma requires the Zvfbfmin extension and the Zfbfmin
>> extension."
>>   1.1 In Embedded    Processor: Zvfbfwma -> Zvfbfmin -> Zve32f
>>   1.2 In Application Processor: Zvfbfwma -> Zvfbfmin -> V
>>   1.3 In both scenarios, there are: Zvfbfwma -> Zfbfmin
>>
>> 2 Depending on different usage scenarios, the Zvfbfwma extension may
>> depend on 'V' or 'Zve32f'. This patch only implements dependencies in
>> scenario of Embedded Processor. This is consistent with the processing
>> strategy in Zvfbfmin. In scenario of Application Processor, it is
>> necessary to explicitly indicate the dependent 'V' extension.
>>
>> For relevant information in gcc, please refer to:
>> <
>> https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=38dd4e26e07c6be7cf4d169141ee4f3a03f3a09d
>> >
>>
>> bfd/ChangeLog:
>>
>>         * elfxx-riscv.c (riscv_multi_subset_supports): Handle Zvfbfwma.
>>         (riscv_multi_subset_supports_ext): Ditto.
>>
>> gas/ChangeLog:
>>
>>         * NEWS: Updated.
>>         * testsuite/gas/riscv/march-help.l: Ditto.
>>         * testsuite/gas/riscv/zvfbfwma-rv32.d: New test.
>>         * testsuite/gas/riscv/zvfbfwma-rv32.s: New test.
>>         * testsuite/gas/riscv/zvfbfwma-rv64.d: New test.
>>         * testsuite/gas/riscv/zvfbfwma-rv64.s: New test.
>>
>> include/ChangeLog:
>>
>>         * opcode/riscv-opc.h (MATCH_VFWMACCBF16_VF): Define.
>>         (MASK_VFWMACCBF16_VF): Ditto.
>>         (MATCH_VFWMACCBF16_VV): Ditto.
>>         (MASK_VFWMACCBF16_VV): Ditto.
>>         (DECLARE_INSN): New declarations for Zvfbfwma.
>>         * opcode/riscv.h (enum riscv_insn_class): Add
>>         INSN_CLASS_ZVFBFWMA
>>
>> opcodes/ChangeLog:
>>
>>         * riscv-opc.c: Add Zvfbfwma instructions.
>> ---
>>  bfd/elfxx-riscv.c                       |  7 +++++++
>>  gas/NEWS                                |  2 ++
>>  gas/testsuite/gas/riscv/march-help.l    |  1 +
>>  gas/testsuite/gas/riscv/zvfbfwma-rv32.d | 12 ++++++++++++
>>  gas/testsuite/gas/riscv/zvfbfwma-rv32.s |  7 +++++++
>>  gas/testsuite/gas/riscv/zvfbfwma-rv64.d | 12 ++++++++++++
>>  gas/testsuite/gas/riscv/zvfbfwma-rv64.s |  7 +++++++
>>  include/opcode/riscv-opc.h              |  8 ++++++++
>>  include/opcode/riscv.h                  |  1 +
>>  opcodes/riscv-opc.c                     |  4 ++++
>>  10 files changed, 61 insertions(+)
>>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv32.d
>>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv32.s
>>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv64.d
>>  create mode 100644 gas/testsuite/gas/riscv/zvfbfwma-rv64.s
>>
>> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
>> index 3d303f02b58..1182c25cab2 100644
>> --- a/bfd/elfxx-riscv.c
>> +++ b/bfd/elfxx-riscv.c
>> @@ -1193,6 +1193,8 @@ static struct riscv_implicit_subset
>> riscv_implicit_subsets[] =
>>    {"v", "zvl128b",     check_implicit_always},
>>    {"zabha", "a",       check_implicit_always},
>>    {"zvfbfmin", "zve32f",       check_implicit_always},
>> +  {"zvfbfwma", "zve32f",       check_implicit_always},
>> +  {"zvfbfwma", "zfbfmin",      check_implicit_always},
>>    {"zvfh", "zvfhmin",  check_implicit_always},
>>    {"zvfh", "zfhmin",   check_implicit_always},
>>    {"zvfhmin", "zve32f",        check_implicit_always},
>> @@ -1396,6 +1398,7 @@ static struct riscv_supported_ext
>> riscv_supported_std_z_ext[] =
>>    {"zvbc",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>>    {"zvfh",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>>    {"zvfbfmin",         ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>> +  {"zvfbfwma",         ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>>    {"zvfhmin",          ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>>    {"zvkb",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>>    {"zvkg",             ISA_SPEC_CLASS_DRAFT,           1, 0,  0 },
>> @@ -2648,6 +2651,8 @@ riscv_multi_subset_supports (riscv_parse_subset_t
>> *rps,
>>        return riscv_subset_supports (rps, "zvbc");
>>      case INSN_CLASS_ZVFBFMIN:
>>        return riscv_subset_supports (rps, "zvfbfmin");
>> +    case INSN_CLASS_ZVFBFWMA:
>> +      return riscv_subset_supports (rps, "zvfbfwma");
>>      case INSN_CLASS_ZVKB:
>>        return riscv_subset_supports (rps, "zvkb");
>>      case INSN_CLASS_ZVKG:
>> @@ -2914,6 +2919,8 @@ riscv_multi_subset_supports_ext
>> (riscv_parse_subset_t *rps,
>>        return _("zvbc");
>>      case INSN_CLASS_ZVFBFMIN:
>>        return "zvfbfmin";
>> +    case INSN_CLASS_ZVFBFWMA:
>> +      return "zvfbfwma";
>>      case INSN_CLASS_ZVKB:
>>        return _("zvkb");
>>      case INSN_CLASS_ZVKG:
>> diff --git a/gas/NEWS b/gas/NEWS
>> index 2c75966d0ce..45eafeff38f 100644
>> --- a/gas/NEWS
>> +++ b/gas/NEWS
>> @@ -1,5 +1,7 @@
>>  -*- text -*-
>>
>> +* Add support for RISC-V Zvfbfwma extension with version 1.0.
>> +
>>  * Add support for RISC-V Zvfbfmin extension with version 1.0.
>>
>>  * Add support for RISC-V Zfbfmin extension with version 1.0.
>> diff --git a/gas/testsuite/gas/riscv/march-help.l
>> b/gas/testsuite/gas/riscv/march-help.l
>> index 1a2ac1eaa08..4b051b189b4 100644
>> --- a/gas/testsuite/gas/riscv/march-help.l
>> +++ b/gas/testsuite/gas/riscv/march-help.l
>> @@ -60,6 +60,7 @@ All available -march extensions for RISC-V:
>>         zvbc                                    1.0
>>         zvfh                                    1.0
>>         zvfbfmin                                1.0
>> +       zvfbfwma                                1.0
>>         zvfhmin                                 1.0
>>         zvkb                                    1.0
>>         zvkg                                    1.0
>> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
>> b/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
>> new file mode 100644
>> index 00000000000..2c00dabc32a
>> --- /dev/null
>> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
>> @@ -0,0 +1,12 @@
>> +#as: -march=rv32i_zvfbfwma
>> +#objdump: -d
>> +
>> +.*:[   ]+file format .*
>> +
>> +Disassembly of section .text:
>> +
>> +0+000 <target>:
>> +[      ]+[0-9a-f]+:[   ]+ee865257[     ]+vfwmaccbf16.vf[       ]+v4,fa2,v8
>> +[      ]+[0-9a-f]+:[   ]+ec865257[     ]+vfwmaccbf16.vf[
>>  ]+v4,fa2,v8,v0.t
>> +[      ]+[0-9a-f]+:[   ]+ee861257[     ]+vfwmaccbf16.vv[       ]+v4,v12,v8
>> +[      ]+[0-9a-f]+:[   ]+ec861257[     ]+vfwmaccbf16.vv[
>>  ]+v4,v12,v8,v0.t
>> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
>> b/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
>> new file mode 100644
>> index 00000000000..f824af98361
>> --- /dev/null
>> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
>> @@ -0,0 +1,7 @@
>> +target:
>> +       # vfwmaccbf16.vf
>> +       vfwmaccbf16.vf v4, fa2, v8
>> +       vfwmaccbf16.vf v4, fa2, v8, v0.t
>> +       # vfwmaccbf16.vv
>> +       vfwmaccbf16.vv v4, v12, v8
>> +       vfwmaccbf16.vv v4, v12, v8, v0.t
>>
>
>No need to seperate rv32 and rv64 test cases. 
Fixed.

>
>
>> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
>> b/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
>> new file mode 100644
>> index 00000000000..05da1328eea
>> --- /dev/null
>> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
>> @@ -0,0 +1,12 @@
>> +#as: -march=rv64iv_zvfbfwma
>> +#objdump: -d
>> +
>> +.*:[   ]+file format .*
>> +
>> +Disassembly of section .text:
>> +
>> +0+000 <target>:
>> +[      ]+[0-9a-f]+:[   ]+ee865257[     ]+vfwmaccbf16.vf[       ]+v4,fa2,v8
>> +[      ]+[0-9a-f]+:[   ]+ec865257[     ]+vfwmaccbf16.vf[
>>  ]+v4,fa2,v8,v0.t
>> +[      ]+[0-9a-f]+:[   ]+ee861257[     ]+vfwmaccbf16.vv[       ]+v4,v12,v8
>> +[      ]+[0-9a-f]+:[   ]+ec861257[     ]+vfwmaccbf16.vv[
>>  ]+v4,v12,v8,v0.t
>> diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
>> b/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
>> new file mode 100644
>> index 00000000000..f824af98361
>> --- /dev/null
>> +++ b/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
>> @@ -0,0 +1,7 @@
>> +target:
>> +       # vfwmaccbf16.vf
>> +       vfwmaccbf16.vf v4, fa2, v8
>> +       vfwmaccbf16.vf v4, fa2, v8, v0.t
>> +       # vfwmaccbf16.vv
>> +       vfwmaccbf16.vv v4, v12, v8
>> +       vfwmaccbf16.vv v4, v12, v8, v0.t
>> diff --git a/include/opcode/riscv-opc.h b/include/opcode/riscv-opc.h
>> index 32b971fb2b3..7db3dd1a1ed 100644
>> --- a/include/opcode/riscv-opc.h
>> +++ b/include/opcode/riscv-opc.h
>> @@ -2375,6 +2375,11 @@
>>  #define MASK_VFNCVTBF16_F_F_W 0xfc0ff07f
>>  #define MATCH_VFWCVTBF16_F_F_V 0x48069057
>>  #define MASK_VFWCVTBF16_F_F_V 0xfc0ff07f
>> +/* Zvfbfwma intructions.  */
>> +#define MATCH_VFWMACCBF16_VF 0xec005057
>> +#define MASK_VFWMACCBF16_VF 0xfc00707f
>> +#define MATCH_VFWMACCBF16_VV 0xec001057
>> +#define MASK_VFWMACCBF16_VV 0xfc00707f
>>  /* Vendor-specific (CORE-V) Xcvmac instructions.  */
>>  #define MATCH_CV_MAC       0x9000302b
>>  #define MASK_CV_MAC        0xfe00707f
>> @@ -3928,6 +3933,9 @@ DECLARE_INSN(FCVT_S_BF16, MATCH_FCVT_S_BF16,
>> MASK_FCVT_S_BF16)
>>  /* Zvfbfmin instructions.  */
>>  DECLARE_INSN(VFNCVTBF16_F_F_W, MATCH_VFNCVTBF16_F_F_W,
>> MASK_VFNCVTBF16_F_F_W)
>>  DECLARE_INSN(VFWCVTBF16_F_F_V, MATCH_VFWCVTBF16_F_F_V,
>> MASK_VFWCVTBF16_F_F_V)
>> +/* Zvfbfwma instructions.  */
>> +DECLARE_INSN(VFWMACCBF16_VF, MATCH_VFWMACCBF16_VF, MASK_VFWMACCBF16_VF)
>> +DECLARE_INSN(VFWMACCBF16_VV, MATCH_VFWMACCBF16_VV, MASK_VFWMACCBF16_VV)
>>  /* Zvbb/Zvkb instructions.  */
>>  DECLARE_INSN(vandn_vv, MATCH_VANDN_VV, MASK_VANDN_VV)
>>  DECLARE_INSN(vandn_vx, MATCH_VANDN_VX, MASK_VANDN_VX)
>> diff --git a/include/opcode/riscv.h b/include/opcode/riscv.h
>> index 4d21b6c3926..1ebfc5210ca 100644
>> --- a/include/opcode/riscv.h
>> +++ b/include/opcode/riscv.h
>> @@ -472,6 +472,7 @@ enum riscv_insn_class
>>    INSN_CLASS_ZVBB,
>>    INSN_CLASS_ZVBC,
>>    INSN_CLASS_ZVFBFMIN,
>> +  INSN_CLASS_ZVFBFWMA,
>>    INSN_CLASS_ZVKB,
>>    INSN_CLASS_ZVKG,
>>    INSN_CLASS_ZVKNED,
>> diff --git a/opcodes/riscv-opc.c b/opcodes/riscv-opc.c
>> index 0a470aee7cc..670e986978c 100644
>> --- a/opcodes/riscv-opc.c
>> +++ b/opcodes/riscv-opc.c
>> @@ -2045,6 +2045,10 @@ const struct riscv_opcode riscv_opcodes[] =
>>  {"vfncvtbf16.f.f.w", 0, INSN_CLASS_ZVFBFMIN, "Vd,VtVm",
>> MATCH_VFNCVTBF16_F_F_W, MASK_VFNCVTBF16_F_F_W, match_opcode, 0},
>>  {"vfwcvtbf16.f.f.v", 0, INSN_CLASS_ZVFBFMIN, "Vd,VtVm",
>> MATCH_VFWCVTBF16_F_F_V, MASK_VFWCVTBF16_F_F_V, match_opcode, 0},
>>
>> +/* Zvfbfwma instructions.  */
>> +{"vfwmaccbf16.vf",  0, INSN_CLASS_ZVFBFWMA, "Vd,S,VtVm",
>> MATCH_VFWMACCBF16_VF, MASK_VFWMACCBF16_VF, match_opcode, 0},
>> +{"vfwmaccbf16.vv",  0, INSN_CLASS_ZVFBFWMA, "Vd,Vs,VtVm",
>> MATCH_VFWMACCBF16_VV, MASK_VFWMACCBF16_VV, match_opcode, 0},
>> +
>>  /* Zvkg instructions.  */
>>  {"vghsh.vv",   0, INSN_CLASS_ZVKG, "Vd,Vt,Vs", MATCH_VGHSH_VV,
>> MASK_VGHSH_VV, match_opcode, 0},
>>  {"vgmul.vv",   0, INSN_CLASS_ZVKG, "Vd,Vt", MATCH_VGMUL_VV,
>> MASK_VGMUL_VV, match_opcode, 0},
>> --
>> 2.17.1
>>
>>
Thanks
Xiao Zeng
  

Patch

diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
index 3d303f02b58..1182c25cab2 100644
--- a/bfd/elfxx-riscv.c
+++ b/bfd/elfxx-riscv.c
@@ -1193,6 +1193,8 @@  static struct riscv_implicit_subset riscv_implicit_subsets[] =
   {"v", "zvl128b",	check_implicit_always},
   {"zabha", "a",	check_implicit_always},
   {"zvfbfmin", "zve32f",	check_implicit_always},
+  {"zvfbfwma", "zve32f",	check_implicit_always},
+  {"zvfbfwma", "zfbfmin",	check_implicit_always},
   {"zvfh", "zvfhmin",	check_implicit_always},
   {"zvfh", "zfhmin",	check_implicit_always},
   {"zvfhmin", "zve32f",	check_implicit_always},
@@ -1396,6 +1398,7 @@  static struct riscv_supported_ext riscv_supported_std_z_ext[] =
   {"zvbc",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zvfh",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zvfbfmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
+  {"zvfbfwma",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zvfhmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zvkb",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zvkg",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
@@ -2648,6 +2651,8 @@  riscv_multi_subset_supports (riscv_parse_subset_t *rps,
       return riscv_subset_supports (rps, "zvbc");
     case INSN_CLASS_ZVFBFMIN:
       return riscv_subset_supports (rps, "zvfbfmin");
+    case INSN_CLASS_ZVFBFWMA:
+      return riscv_subset_supports (rps, "zvfbfwma");
     case INSN_CLASS_ZVKB:
       return riscv_subset_supports (rps, "zvkb");
     case INSN_CLASS_ZVKG:
@@ -2914,6 +2919,8 @@  riscv_multi_subset_supports_ext (riscv_parse_subset_t *rps,
       return _("zvbc");
     case INSN_CLASS_ZVFBFMIN:
       return "zvfbfmin";
+    case INSN_CLASS_ZVFBFWMA:
+      return "zvfbfwma";
     case INSN_CLASS_ZVKB:
       return _("zvkb");
     case INSN_CLASS_ZVKG:
diff --git a/gas/NEWS b/gas/NEWS
index 2c75966d0ce..45eafeff38f 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -1,5 +1,7 @@ 
 -*- text -*-
 
+* Add support for RISC-V Zvfbfwma extension with version 1.0.
+
 * Add support for RISC-V Zvfbfmin extension with version 1.0.
 
 * Add support for RISC-V Zfbfmin extension with version 1.0.
diff --git a/gas/testsuite/gas/riscv/march-help.l b/gas/testsuite/gas/riscv/march-help.l
index 1a2ac1eaa08..4b051b189b4 100644
--- a/gas/testsuite/gas/riscv/march-help.l
+++ b/gas/testsuite/gas/riscv/march-help.l
@@ -60,6 +60,7 @@  All available -march extensions for RISC-V:
 	zvbc                                    1.0
 	zvfh                                    1.0
 	zvfbfmin                                1.0
+	zvfbfwma                                1.0
 	zvfhmin                                 1.0
 	zvkb                                    1.0
 	zvkg                                    1.0
diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv32.d b/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
new file mode 100644
index 00000000000..2c00dabc32a
--- /dev/null
+++ b/gas/testsuite/gas/riscv/zvfbfwma-rv32.d
@@ -0,0 +1,12 @@ 
+#as: -march=rv32i_zvfbfwma
+#objdump: -d
+
+.*:[ 	]+file format .*
+
+Disassembly of section .text:
+
+0+000 <target>:
+[ 	]+[0-9a-f]+:[ 	]+ee865257[ 	]+vfwmaccbf16.vf[ 	]+v4,fa2,v8
+[ 	]+[0-9a-f]+:[ 	]+ec865257[ 	]+vfwmaccbf16.vf[ 	]+v4,fa2,v8,v0.t
+[ 	]+[0-9a-f]+:[ 	]+ee861257[ 	]+vfwmaccbf16.vv[ 	]+v4,v12,v8
+[ 	]+[0-9a-f]+:[ 	]+ec861257[ 	]+vfwmaccbf16.vv[ 	]+v4,v12,v8,v0.t
diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv32.s b/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
new file mode 100644
index 00000000000..f824af98361
--- /dev/null
+++ b/gas/testsuite/gas/riscv/zvfbfwma-rv32.s
@@ -0,0 +1,7 @@ 
+target:
+	# vfwmaccbf16.vf
+	vfwmaccbf16.vf v4, fa2, v8
+	vfwmaccbf16.vf v4, fa2, v8, v0.t
+	# vfwmaccbf16.vv
+	vfwmaccbf16.vv v4, v12, v8
+	vfwmaccbf16.vv v4, v12, v8, v0.t
diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv64.d b/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
new file mode 100644
index 00000000000..05da1328eea
--- /dev/null
+++ b/gas/testsuite/gas/riscv/zvfbfwma-rv64.d
@@ -0,0 +1,12 @@ 
+#as: -march=rv64iv_zvfbfwma
+#objdump: -d
+
+.*:[ 	]+file format .*
+
+Disassembly of section .text:
+
+0+000 <target>:
+[ 	]+[0-9a-f]+:[ 	]+ee865257[ 	]+vfwmaccbf16.vf[ 	]+v4,fa2,v8
+[ 	]+[0-9a-f]+:[ 	]+ec865257[ 	]+vfwmaccbf16.vf[ 	]+v4,fa2,v8,v0.t
+[ 	]+[0-9a-f]+:[ 	]+ee861257[ 	]+vfwmaccbf16.vv[ 	]+v4,v12,v8
+[ 	]+[0-9a-f]+:[ 	]+ec861257[ 	]+vfwmaccbf16.vv[ 	]+v4,v12,v8,v0.t
diff --git a/gas/testsuite/gas/riscv/zvfbfwma-rv64.s b/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
new file mode 100644
index 00000000000..f824af98361
--- /dev/null
+++ b/gas/testsuite/gas/riscv/zvfbfwma-rv64.s
@@ -0,0 +1,7 @@ 
+target:
+	# vfwmaccbf16.vf
+	vfwmaccbf16.vf v4, fa2, v8
+	vfwmaccbf16.vf v4, fa2, v8, v0.t
+	# vfwmaccbf16.vv
+	vfwmaccbf16.vv v4, v12, v8
+	vfwmaccbf16.vv v4, v12, v8, v0.t
diff --git a/include/opcode/riscv-opc.h b/include/opcode/riscv-opc.h
index 32b971fb2b3..7db3dd1a1ed 100644
--- a/include/opcode/riscv-opc.h
+++ b/include/opcode/riscv-opc.h
@@ -2375,6 +2375,11 @@ 
 #define MASK_VFNCVTBF16_F_F_W 0xfc0ff07f
 #define MATCH_VFWCVTBF16_F_F_V 0x48069057
 #define MASK_VFWCVTBF16_F_F_V 0xfc0ff07f
+/* Zvfbfwma intructions.  */
+#define MATCH_VFWMACCBF16_VF 0xec005057
+#define MASK_VFWMACCBF16_VF 0xfc00707f
+#define MATCH_VFWMACCBF16_VV 0xec001057
+#define MASK_VFWMACCBF16_VV 0xfc00707f
 /* Vendor-specific (CORE-V) Xcvmac instructions.  */
 #define MATCH_CV_MAC       0x9000302b
 #define MASK_CV_MAC        0xfe00707f
@@ -3928,6 +3933,9 @@  DECLARE_INSN(FCVT_S_BF16, MATCH_FCVT_S_BF16, MASK_FCVT_S_BF16)
 /* Zvfbfmin instructions.  */
 DECLARE_INSN(VFNCVTBF16_F_F_W, MATCH_VFNCVTBF16_F_F_W, MASK_VFNCVTBF16_F_F_W)
 DECLARE_INSN(VFWCVTBF16_F_F_V, MATCH_VFWCVTBF16_F_F_V, MASK_VFWCVTBF16_F_F_V)
+/* Zvfbfwma instructions.  */
+DECLARE_INSN(VFWMACCBF16_VF, MATCH_VFWMACCBF16_VF, MASK_VFWMACCBF16_VF)
+DECLARE_INSN(VFWMACCBF16_VV, MATCH_VFWMACCBF16_VV, MASK_VFWMACCBF16_VV)
 /* Zvbb/Zvkb instructions.  */
 DECLARE_INSN(vandn_vv, MATCH_VANDN_VV, MASK_VANDN_VV)
 DECLARE_INSN(vandn_vx, MATCH_VANDN_VX, MASK_VANDN_VX)
diff --git a/include/opcode/riscv.h b/include/opcode/riscv.h
index 4d21b6c3926..1ebfc5210ca 100644
--- a/include/opcode/riscv.h
+++ b/include/opcode/riscv.h
@@ -472,6 +472,7 @@  enum riscv_insn_class
   INSN_CLASS_ZVBB,
   INSN_CLASS_ZVBC,
   INSN_CLASS_ZVFBFMIN,
+  INSN_CLASS_ZVFBFWMA,
   INSN_CLASS_ZVKB,
   INSN_CLASS_ZVKG,
   INSN_CLASS_ZVKNED,
diff --git a/opcodes/riscv-opc.c b/opcodes/riscv-opc.c
index 0a470aee7cc..670e986978c 100644
--- a/opcodes/riscv-opc.c
+++ b/opcodes/riscv-opc.c
@@ -2045,6 +2045,10 @@  const struct riscv_opcode riscv_opcodes[] =
 {"vfncvtbf16.f.f.w", 0, INSN_CLASS_ZVFBFMIN, "Vd,VtVm", MATCH_VFNCVTBF16_F_F_W, MASK_VFNCVTBF16_F_F_W, match_opcode, 0},
 {"vfwcvtbf16.f.f.v", 0, INSN_CLASS_ZVFBFMIN, "Vd,VtVm", MATCH_VFWCVTBF16_F_F_V, MASK_VFWCVTBF16_F_F_V, match_opcode, 0},
 
+/* Zvfbfwma instructions.  */
+{"vfwmaccbf16.vf",  0, INSN_CLASS_ZVFBFWMA, "Vd,S,VtVm",  MATCH_VFWMACCBF16_VF, MASK_VFWMACCBF16_VF, match_opcode, 0},
+{"vfwmaccbf16.vv",  0, INSN_CLASS_ZVFBFWMA, "Vd,Vs,VtVm", MATCH_VFWMACCBF16_VV, MASK_VFWMACCBF16_VV, match_opcode, 0},
+
 /* Zvkg instructions.  */
 {"vghsh.vv",   0, INSN_CLASS_ZVKG, "Vd,Vt,Vs", MATCH_VGHSH_VV, MASK_VGHSH_VV, match_opcode, 0},
 {"vgmul.vv",   0, INSN_CLASS_ZVKG, "Vd,Vt", MATCH_VGMUL_VV, MASK_VGMUL_VV, match_opcode, 0},