public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] RISC-V: Support Zfbfmin extension.
@ 2024-06-04 11:09 Jiawei
  2024-06-04 16:52 ` Jan Beulich
  0 siblings, 1 reply; 3+ messages in thread
From: Jiawei @ 2024-06-04 11:09 UTC (permalink / raw)
  To: binutils; +Cc: nelson, kito.cheng, palmer, jeffreyalaw, Jiawei

This patch support RISC-V Zfbfmin extension with two new instructions.

bfd/ChangeLog:

	* elfxx-riscv.c (riscv_multi_subset_supports): New extension.
	(riscv_multi_subset_supports_ext): Ditto.

gas/ChangeLog:

	* testsuite/gas/riscv/march-help.l: New extension.
	* testsuite/gas/riscv/zfbfmin.d: New test.
	* testsuite/gas/riscv/zfbfmin.s: New test.

include/ChangeLog:

	* opcode/riscv-opc.h (MATCH_FCVT_BF16_S): New opcode.
	(MASK_FCVT_BF16_S): Ditto.
	(MATCH_FCVT_S_BF16): Ditto.
	(MASK_FCVT_S_BF16): Ditto.
	(DECLARE_INSN): New instructions.
	* opcode/riscv.h (enum riscv_insn_class): New extension.

opcodes/ChangeLog:

	* riscv-opc.c: New instructions.

---
 bfd/elfxx-riscv.c                    |  7 +++++++
 gas/testsuite/gas/riscv/march-help.l |  1 +
 gas/testsuite/gas/riscv/zfbfmin.d    | 11 +++++++++++
 gas/testsuite/gas/riscv/zfbfmin.s    |  4 ++++
 include/opcode/riscv-opc.h           |  6 ++++++
 include/opcode/riscv.h               |  1 +
 opcodes/riscv-opc.c                  |  5 +++++
 7 files changed, 35 insertions(+)
 create mode 100644 gas/testsuite/gas/riscv/zfbfmin.d
 create mode 100644 gas/testsuite/gas/riscv/zfbfmin.s

diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
index dfacb87eda0..f203237420a 100644
--- a/bfd/elfxx-riscv.c
+++ b/bfd/elfxx-riscv.c
@@ -1226,6 +1226,8 @@ static struct riscv_implicit_subset riscv_implicit_subsets[] =
   {"d", "f",		check_implicit_always},
   {"zfh", "zfhmin",	check_implicit_always},
   {"zfhmin", "f",	check_implicit_always},
+  {"zfbfmin", "zfhmin",	check_implicit_always},
+  {"zfbfmin", "f",	check_implicit_always},
   {"f", "zicsr",	check_implicit_always},
   {"zqinx", "zdinx",	check_implicit_always},
   {"zdinx", "zfinx",	check_implicit_always},
@@ -1362,6 +1364,7 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] =
   {"zfa",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zfh",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zfhmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
+  {"zfbfmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zfinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zdinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
   {"zqinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
@@ -2579,6 +2582,8 @@ riscv_multi_subset_supports (riscv_parse_subset_t *rps,
 	       && riscv_subset_supports (rps, "q"))
 	      || (riscv_subset_supports (rps, "zhinxmin")
 		  && riscv_subset_supports (rps, "zqinx")));
+    case INSN_CLASS_ZFBFMIN:
+      return riscv_subset_supports (rps, "zfbfmin");
     case INSN_CLASS_ZFA:
       return riscv_subset_supports (rps, "zfa");
     case INSN_CLASS_D_AND_ZFA:
@@ -2827,6 +2832,8 @@ riscv_multi_subset_supports_ext (riscv_parse_subset_t *rps,
 	return "zhinxmin";
       else
 	return _("zfhmin' and `q', or `zhinxmin' and `zqinx");
+    case INSN_CLASS_ZFBFMIN:
+      return "zfbfmin";
     case INSN_CLASS_ZFA:
       return "zfa";
     case INSN_CLASS_D_AND_ZFA:
diff --git a/gas/testsuite/gas/riscv/march-help.l b/gas/testsuite/gas/riscv/march-help.l
index c5754837e05..ee5a3f03d90 100644
--- a/gas/testsuite/gas/riscv/march-help.l
+++ b/gas/testsuite/gas/riscv/march-help.l
@@ -28,6 +28,7 @@ All available -march extensions for RISC-V:
 	zfa                                     1.0
 	zfh                                     1.0
 	zfhmin                                  1.0
+	zfbfmin                                 1.0
 	zfinx                                   1.0
 	zdinx                                   1.0
 	zqinx                                   1.0
diff --git a/gas/testsuite/gas/riscv/zfbfmin.d b/gas/testsuite/gas/riscv/zfbfmin.d
new file mode 100644
index 00000000000..b04e6e030db
--- /dev/null
+++ b/gas/testsuite/gas/riscv/zfbfmin.d
@@ -0,0 +1,11 @@
+#as: -march=rv64i_zfbfmin
+#objdump: -dr
+
+.*:[ 	]+file format .*
+
+Disassembly of section .text:
+
+0+000 <target>:
+[ 	]+[0-9a-f]+:[ 	]+40658553[ 	]+fcvt.s.bf16[ 	]+fa0,fa1
+[ 	]+[0-9a-f]+:[ 	]+4485f553[ 	]+fcvt.bf16.s[ 	]+fa0,fa1
+[ 	]+[0-9a-f]+:[ 	]+44858553[ 	]+fcvt.bf16.s[ 	]+fa0,fa1,rne
diff --git a/gas/testsuite/gas/riscv/zfbfmin.s b/gas/testsuite/gas/riscv/zfbfmin.s
new file mode 100644
index 00000000000..ddf873f611a
--- /dev/null
+++ b/gas/testsuite/gas/riscv/zfbfmin.s
@@ -0,0 +1,4 @@
+target:
+	fcvt.s.bf16	fa0, fa1
+	fcvt.bf16.s	fa0, fa1
+	fcvt.bf16.s	fa0, fa1, rne
diff --git a/include/opcode/riscv-opc.h b/include/opcode/riscv-opc.h
index ae14e14d427..e33b81bf4fb 100644
--- a/include/opcode/riscv-opc.h
+++ b/include/opcode/riscv-opc.h
@@ -821,8 +821,12 @@
 #define MASK_FMIN_H 0xfe00707f
 #define MATCH_FMAX_H 0x2c001053
 #define MASK_FMAX_H 0xfe00707f
+#define MATCH_FCVT_BF16_S 0x44800053
+#define MASK_FCVT_BF16_S 0xfff0007f
 #define MATCH_FCVT_H_S 0x44000053
 #define MASK_FCVT_H_S 0xfff0007f
+#define MATCH_FCVT_S_BF16 0x40600053
+#define MASK_FCVT_S_BF16 0xfff0007f
 #define MATCH_FCVT_S_H 0x40200053
 #define MASK_FCVT_S_H 0xfff0007f
 #define MATCH_FSQRT_H 0x5c000053
@@ -3675,6 +3679,8 @@ DECLARE_INSN(fmin_d, MATCH_FMIN_D, MASK_FMIN_D)
 DECLARE_INSN(fmax_d, MATCH_FMAX_D, MASK_FMAX_D)
 DECLARE_INSN(fcvt_s_d, MATCH_FCVT_S_D, MASK_FCVT_S_D)
 DECLARE_INSN(fcvt_d_s, MATCH_FCVT_D_S, MASK_FCVT_D_S)
+DECLARE_INSN(fcvt_s_bf16, MATCH_FCVT_S_BF16 MASK_FCVT_S_BF16)
+DECLARE_INSN(fcvt_bf16_s, MATCH_FCVT_BF16_S, MASK_FCVT_BF16_S)
 DECLARE_INSN(fsqrt_d, MATCH_FSQRT_D, MASK_FSQRT_D)
 DECLARE_INSN(fadd_q, MATCH_FADD_Q, MASK_FADD_Q)
 DECLARE_INSN(fsub_q, MATCH_FSUB_Q, MASK_FSUB_Q)
diff --git a/include/opcode/riscv.h b/include/opcode/riscv.h
index 5f516a1026e..e10b2b890b5 100644
--- a/include/opcode/riscv.h
+++ b/include/opcode/riscv.h
@@ -446,6 +446,7 @@ enum riscv_insn_class
   INSN_CLASS_ZFHMIN_INX,
   INSN_CLASS_ZFHMIN_AND_D_INX,
   INSN_CLASS_ZFHMIN_AND_Q_INX,
+  INSN_CLASS_ZFBFMIN,
   INSN_CLASS_ZFA,
   INSN_CLASS_D_AND_ZFA,
   INSN_CLASS_Q_AND_ZFA,
diff --git a/opcodes/riscv-opc.c b/opcodes/riscv-opc.c
index 1ef4eaddf4d..f73d2c7a81d 100644
--- a/opcodes/riscv-opc.c
+++ b/opcodes/riscv-opc.c
@@ -764,6 +764,11 @@ const struct riscv_opcode riscv_opcodes[] =
 {"remw",      64, INSN_CLASS_M,     "d,s,t",     MATCH_REMW, MASK_REMW, match_opcode, 0 },
 {"remuw",     64, INSN_CLASS_M,     "d,s,t",     MATCH_REMUW, MASK_REMUW, match_opcode, 0 },
 
+/* BF16 floating-point instruction subset.  */
+{"fcvt.bf16.s",   0, INSN_CLASS_ZFBFMIN, "D,S",     MATCH_FCVT_BF16_S|MASK_RM, MASK_FCVT_BF16_S|MASK_RM, match_opcode, 0 },
+{"fcvt.bf16.s",   0, INSN_CLASS_ZFBFMIN, "D,S,m",   MATCH_FCVT_BF16_S, MASK_FCVT_BF16_S, match_opcode, 0 },
+{"fcvt.s.bf16",   0, INSN_CLASS_ZFBFMIN, "D,S",     MATCH_FCVT_S_BF16, MASK_FCVT_S_BF16|MASK_RM, match_opcode, 0 },
+
 /* Half-precision floating-point instruction subset.  */
 {"flh",        0, INSN_CLASS_ZFHMIN,   "D,o(s)",    MATCH_FLH, MASK_FLH, match_opcode, INSN_DREF|INSN_2_BYTE },
 {"flh",        0, INSN_CLASS_ZFHMIN,   "D,A,s",     0, (int) M_FLx, match_rs1_nonzero, INSN_MACRO },
-- 
2.25.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] RISC-V: Support Zfbfmin extension.
  2024-06-04 11:09 [PATCH] RISC-V: Support Zfbfmin extension Jiawei
@ 2024-06-04 16:52 ` Jan Beulich
  2024-06-05  0:25   ` Jiawei
  0 siblings, 1 reply; 3+ messages in thread
From: Jan Beulich @ 2024-06-04 16:52 UTC (permalink / raw)
  To: Jiawei; +Cc: nelson, kito.cheng, palmer, jeffreyalaw, binutils

On 04.06.2024 13:09, Jiawei wrote:
> This patch support RISC-V Zfbfmin extension with two new instructions.
> 
> bfd/ChangeLog:
> 
> 	* elfxx-riscv.c (riscv_multi_subset_supports): New extension.
> 	(riscv_multi_subset_supports_ext): Ditto.
> 
> gas/ChangeLog:
> 
> 	* testsuite/gas/riscv/march-help.l: New extension.
> 	* testsuite/gas/riscv/zfbfmin.d: New test.
> 	* testsuite/gas/riscv/zfbfmin.s: New test.
> 
> include/ChangeLog:
> 
> 	* opcode/riscv-opc.h (MATCH_FCVT_BF16_S): New opcode.
> 	(MASK_FCVT_BF16_S): Ditto.
> 	(MATCH_FCVT_S_BF16): Ditto.
> 	(MASK_FCVT_S_BF16): Ditto.
> 	(DECLARE_INSN): New instructions.
> 	* opcode/riscv.h (enum riscv_insn_class): New extension.
> 
> opcodes/ChangeLog:
> 
> 	* riscv-opc.c: New instructions.
> 
> ---
>  bfd/elfxx-riscv.c                    |  7 +++++++
>  gas/testsuite/gas/riscv/march-help.l |  1 +
>  gas/testsuite/gas/riscv/zfbfmin.d    | 11 +++++++++++
>  gas/testsuite/gas/riscv/zfbfmin.s    |  4 ++++
>  include/opcode/riscv-opc.h           |  6 ++++++
>  include/opcode/riscv.h               |  1 +
>  opcodes/riscv-opc.c                  |  5 +++++
>  7 files changed, 35 insertions(+)
>  create mode 100644 gas/testsuite/gas/riscv/zfbfmin.d
>  create mode 100644 gas/testsuite/gas/riscv/zfbfmin.s

Just a few days ago there was another submission for, I think,
the same extension:
https://sourceware.org/pipermail/binutils/2024-May/134412.html
yet then in a series covering further related extensions as well.

Jan

> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index dfacb87eda0..f203237420a 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1226,6 +1226,8 @@ static struct riscv_implicit_subset riscv_implicit_subsets[] =
>    {"d", "f",		check_implicit_always},
>    {"zfh", "zfhmin",	check_implicit_always},
>    {"zfhmin", "f",	check_implicit_always},
> +  {"zfbfmin", "zfhmin",	check_implicit_always},
> +  {"zfbfmin", "f",	check_implicit_always},
>    {"f", "zicsr",	check_implicit_always},
>    {"zqinx", "zdinx",	check_implicit_always},
>    {"zdinx", "zfinx",	check_implicit_always},
> @@ -1362,6 +1364,7 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] =
>    {"zfa",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>    {"zfh",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>    {"zfhmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
> +  {"zfbfmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>    {"zfinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>    {"zdinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>    {"zqinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
> @@ -2579,6 +2582,8 @@ riscv_multi_subset_supports (riscv_parse_subset_t *rps,
>  	       && riscv_subset_supports (rps, "q"))
>  	      || (riscv_subset_supports (rps, "zhinxmin")
>  		  && riscv_subset_supports (rps, "zqinx")));
> +    case INSN_CLASS_ZFBFMIN:
> +      return riscv_subset_supports (rps, "zfbfmin");
>      case INSN_CLASS_ZFA:
>        return riscv_subset_supports (rps, "zfa");
>      case INSN_CLASS_D_AND_ZFA:
> @@ -2827,6 +2832,8 @@ riscv_multi_subset_supports_ext (riscv_parse_subset_t *rps,
>  	return "zhinxmin";
>        else
>  	return _("zfhmin' and `q', or `zhinxmin' and `zqinx");
> +    case INSN_CLASS_ZFBFMIN:
> +      return "zfbfmin";
>      case INSN_CLASS_ZFA:
>        return "zfa";
>      case INSN_CLASS_D_AND_ZFA:
> diff --git a/gas/testsuite/gas/riscv/march-help.l b/gas/testsuite/gas/riscv/march-help.l
> index c5754837e05..ee5a3f03d90 100644
> --- a/gas/testsuite/gas/riscv/march-help.l
> +++ b/gas/testsuite/gas/riscv/march-help.l
> @@ -28,6 +28,7 @@ All available -march extensions for RISC-V:
>  	zfa                                     1.0
>  	zfh                                     1.0
>  	zfhmin                                  1.0
> +	zfbfmin                                 1.0
>  	zfinx                                   1.0
>  	zdinx                                   1.0
>  	zqinx                                   1.0
> diff --git a/gas/testsuite/gas/riscv/zfbfmin.d b/gas/testsuite/gas/riscv/zfbfmin.d
> new file mode 100644
> index 00000000000..b04e6e030db
> --- /dev/null
> +++ b/gas/testsuite/gas/riscv/zfbfmin.d
> @@ -0,0 +1,11 @@
> +#as: -march=rv64i_zfbfmin
> +#objdump: -dr
> +
> +.*:[ 	]+file format .*
> +
> +Disassembly of section .text:
> +
> +0+000 <target>:
> +[ 	]+[0-9a-f]+:[ 	]+40658553[ 	]+fcvt.s.bf16[ 	]+fa0,fa1
> +[ 	]+[0-9a-f]+:[ 	]+4485f553[ 	]+fcvt.bf16.s[ 	]+fa0,fa1
> +[ 	]+[0-9a-f]+:[ 	]+44858553[ 	]+fcvt.bf16.s[ 	]+fa0,fa1,rne
> diff --git a/gas/testsuite/gas/riscv/zfbfmin.s b/gas/testsuite/gas/riscv/zfbfmin.s
> new file mode 100644
> index 00000000000..ddf873f611a
> --- /dev/null
> +++ b/gas/testsuite/gas/riscv/zfbfmin.s
> @@ -0,0 +1,4 @@
> +target:
> +	fcvt.s.bf16	fa0, fa1
> +	fcvt.bf16.s	fa0, fa1
> +	fcvt.bf16.s	fa0, fa1, rne
> diff --git a/include/opcode/riscv-opc.h b/include/opcode/riscv-opc.h
> index ae14e14d427..e33b81bf4fb 100644
> --- a/include/opcode/riscv-opc.h
> +++ b/include/opcode/riscv-opc.h
> @@ -821,8 +821,12 @@
>  #define MASK_FMIN_H 0xfe00707f
>  #define MATCH_FMAX_H 0x2c001053
>  #define MASK_FMAX_H 0xfe00707f
> +#define MATCH_FCVT_BF16_S 0x44800053
> +#define MASK_FCVT_BF16_S 0xfff0007f
>  #define MATCH_FCVT_H_S 0x44000053
>  #define MASK_FCVT_H_S 0xfff0007f
> +#define MATCH_FCVT_S_BF16 0x40600053
> +#define MASK_FCVT_S_BF16 0xfff0007f
>  #define MATCH_FCVT_S_H 0x40200053
>  #define MASK_FCVT_S_H 0xfff0007f
>  #define MATCH_FSQRT_H 0x5c000053
> @@ -3675,6 +3679,8 @@ DECLARE_INSN(fmin_d, MATCH_FMIN_D, MASK_FMIN_D)
>  DECLARE_INSN(fmax_d, MATCH_FMAX_D, MASK_FMAX_D)
>  DECLARE_INSN(fcvt_s_d, MATCH_FCVT_S_D, MASK_FCVT_S_D)
>  DECLARE_INSN(fcvt_d_s, MATCH_FCVT_D_S, MASK_FCVT_D_S)
> +DECLARE_INSN(fcvt_s_bf16, MATCH_FCVT_S_BF16 MASK_FCVT_S_BF16)
> +DECLARE_INSN(fcvt_bf16_s, MATCH_FCVT_BF16_S, MASK_FCVT_BF16_S)
>  DECLARE_INSN(fsqrt_d, MATCH_FSQRT_D, MASK_FSQRT_D)
>  DECLARE_INSN(fadd_q, MATCH_FADD_Q, MASK_FADD_Q)
>  DECLARE_INSN(fsub_q, MATCH_FSUB_Q, MASK_FSUB_Q)
> diff --git a/include/opcode/riscv.h b/include/opcode/riscv.h
> index 5f516a1026e..e10b2b890b5 100644
> --- a/include/opcode/riscv.h
> +++ b/include/opcode/riscv.h
> @@ -446,6 +446,7 @@ enum riscv_insn_class
>    INSN_CLASS_ZFHMIN_INX,
>    INSN_CLASS_ZFHMIN_AND_D_INX,
>    INSN_CLASS_ZFHMIN_AND_Q_INX,
> +  INSN_CLASS_ZFBFMIN,
>    INSN_CLASS_ZFA,
>    INSN_CLASS_D_AND_ZFA,
>    INSN_CLASS_Q_AND_ZFA,
> diff --git a/opcodes/riscv-opc.c b/opcodes/riscv-opc.c
> index 1ef4eaddf4d..f73d2c7a81d 100644
> --- a/opcodes/riscv-opc.c
> +++ b/opcodes/riscv-opc.c
> @@ -764,6 +764,11 @@ const struct riscv_opcode riscv_opcodes[] =
>  {"remw",      64, INSN_CLASS_M,     "d,s,t",     MATCH_REMW, MASK_REMW, match_opcode, 0 },
>  {"remuw",     64, INSN_CLASS_M,     "d,s,t",     MATCH_REMUW, MASK_REMUW, match_opcode, 0 },
>  
> +/* BF16 floating-point instruction subset.  */
> +{"fcvt.bf16.s",   0, INSN_CLASS_ZFBFMIN, "D,S",     MATCH_FCVT_BF16_S|MASK_RM, MASK_FCVT_BF16_S|MASK_RM, match_opcode, 0 },
> +{"fcvt.bf16.s",   0, INSN_CLASS_ZFBFMIN, "D,S,m",   MATCH_FCVT_BF16_S, MASK_FCVT_BF16_S, match_opcode, 0 },
> +{"fcvt.s.bf16",   0, INSN_CLASS_ZFBFMIN, "D,S",     MATCH_FCVT_S_BF16, MASK_FCVT_S_BF16|MASK_RM, match_opcode, 0 },
> +
>  /* Half-precision floating-point instruction subset.  */
>  {"flh",        0, INSN_CLASS_ZFHMIN,   "D,o(s)",    MATCH_FLH, MASK_FLH, match_opcode, INSN_DREF|INSN_2_BYTE },
>  {"flh",        0, INSN_CLASS_ZFHMIN,   "D,A,s",     0, (int) M_FLx, match_rs1_nonzero, INSN_MACRO },


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] RISC-V: Support Zfbfmin extension.
  2024-06-04 16:52 ` Jan Beulich
@ 2024-06-05  0:25   ` Jiawei
  0 siblings, 0 replies; 3+ messages in thread
From: Jiawei @ 2024-06-05  0:25 UTC (permalink / raw)
  To: Jan Beulich; +Cc: nelson, kito.cheng, palmer, jeffreyalaw, binutils


在 2024/6/5 0:52, Jan Beulich 写道:
> On 04.06.2024 13:09, Jiawei wrote:
>> This patch support RISC-V Zfbfmin extension with two new instructions.
>>
>> bfd/ChangeLog:
>>
>> 	* elfxx-riscv.c (riscv_multi_subset_supports): New extension.
>> 	(riscv_multi_subset_supports_ext): Ditto.
>>
>> gas/ChangeLog:
>>
>> 	* testsuite/gas/riscv/march-help.l: New extension.
>> 	* testsuite/gas/riscv/zfbfmin.d: New test.
>> 	* testsuite/gas/riscv/zfbfmin.s: New test.
>>
>> include/ChangeLog:
>>
>> 	* opcode/riscv-opc.h (MATCH_FCVT_BF16_S): New opcode.
>> 	(MASK_FCVT_BF16_S): Ditto.
>> 	(MATCH_FCVT_S_BF16): Ditto.
>> 	(MASK_FCVT_S_BF16): Ditto.
>> 	(DECLARE_INSN): New instructions.
>> 	* opcode/riscv.h (enum riscv_insn_class): New extension.
>>
>> opcodes/ChangeLog:
>>
>> 	* riscv-opc.c: New instructions.
>>
>> ---
>>   bfd/elfxx-riscv.c                    |  7 +++++++
>>   gas/testsuite/gas/riscv/march-help.l |  1 +
>>   gas/testsuite/gas/riscv/zfbfmin.d    | 11 +++++++++++
>>   gas/testsuite/gas/riscv/zfbfmin.s    |  4 ++++
>>   include/opcode/riscv-opc.h           |  6 ++++++
>>   include/opcode/riscv.h               |  1 +
>>   opcodes/riscv-opc.c                  |  5 +++++
>>   7 files changed, 35 insertions(+)
>>   create mode 100644 gas/testsuite/gas/riscv/zfbfmin.d
>>   create mode 100644 gas/testsuite/gas/riscv/zfbfmin.s
> Just a few days ago there was another submission for, I think,
> the same extension:
> https://sourceware.org/pipermail/binutils/2024-May/134412.html
> yet then in a series covering further related extensions as well.
>
> Jan

Thanks for your mention, I will go to help review.


BR,

Jiawei

>
>> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
>> index dfacb87eda0..f203237420a 100644
>> --- a/bfd/elfxx-riscv.c
>> +++ b/bfd/elfxx-riscv.c
>> @@ -1226,6 +1226,8 @@ static struct riscv_implicit_subset riscv_implicit_subsets[] =
>>     {"d", "f",		check_implicit_always},
>>     {"zfh", "zfhmin",	check_implicit_always},
>>     {"zfhmin", "f",	check_implicit_always},
>> +  {"zfbfmin", "zfhmin",	check_implicit_always},
>> +  {"zfbfmin", "f",	check_implicit_always},
>>     {"f", "zicsr",	check_implicit_always},
>>     {"zqinx", "zdinx",	check_implicit_always},
>>     {"zdinx", "zfinx",	check_implicit_always},
>> @@ -1362,6 +1364,7 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] =
>>     {"zfa",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>>     {"zfh",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>>     {"zfhmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>> +  {"zfbfmin",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>>     {"zfinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>>     {"zdinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>>     {"zqinx",		ISA_SPEC_CLASS_DRAFT,		1, 0,  0 },
>> @@ -2579,6 +2582,8 @@ riscv_multi_subset_supports (riscv_parse_subset_t *rps,
>>   	       && riscv_subset_supports (rps, "q"))
>>   	      || (riscv_subset_supports (rps, "zhinxmin")
>>   		  && riscv_subset_supports (rps, "zqinx")));
>> +    case INSN_CLASS_ZFBFMIN:
>> +      return riscv_subset_supports (rps, "zfbfmin");
>>       case INSN_CLASS_ZFA:
>>         return riscv_subset_supports (rps, "zfa");
>>       case INSN_CLASS_D_AND_ZFA:
>> @@ -2827,6 +2832,8 @@ riscv_multi_subset_supports_ext (riscv_parse_subset_t *rps,
>>   	return "zhinxmin";
>>         else
>>   	return _("zfhmin' and `q', or `zhinxmin' and `zqinx");
>> +    case INSN_CLASS_ZFBFMIN:
>> +      return "zfbfmin";
>>       case INSN_CLASS_ZFA:
>>         return "zfa";
>>       case INSN_CLASS_D_AND_ZFA:
>> diff --git a/gas/testsuite/gas/riscv/march-help.l b/gas/testsuite/gas/riscv/march-help.l
>> index c5754837e05..ee5a3f03d90 100644
>> --- a/gas/testsuite/gas/riscv/march-help.l
>> +++ b/gas/testsuite/gas/riscv/march-help.l
>> @@ -28,6 +28,7 @@ All available -march extensions for RISC-V:
>>   	zfa                                     1.0
>>   	zfh                                     1.0
>>   	zfhmin                                  1.0
>> +	zfbfmin                                 1.0
>>   	zfinx                                   1.0
>>   	zdinx                                   1.0
>>   	zqinx                                   1.0
>> diff --git a/gas/testsuite/gas/riscv/zfbfmin.d b/gas/testsuite/gas/riscv/zfbfmin.d
>> new file mode 100644
>> index 00000000000..b04e6e030db
>> --- /dev/null
>> +++ b/gas/testsuite/gas/riscv/zfbfmin.d
>> @@ -0,0 +1,11 @@
>> +#as: -march=rv64i_zfbfmin
>> +#objdump: -dr
>> +
>> +.*:[ 	]+file format .*
>> +
>> +Disassembly of section .text:
>> +
>> +0+000 <target>:
>> +[ 	]+[0-9a-f]+:[ 	]+40658553[ 	]+fcvt.s.bf16[ 	]+fa0,fa1
>> +[ 	]+[0-9a-f]+:[ 	]+4485f553[ 	]+fcvt.bf16.s[ 	]+fa0,fa1
>> +[ 	]+[0-9a-f]+:[ 	]+44858553[ 	]+fcvt.bf16.s[ 	]+fa0,fa1,rne
>> diff --git a/gas/testsuite/gas/riscv/zfbfmin.s b/gas/testsuite/gas/riscv/zfbfmin.s
>> new file mode 100644
>> index 00000000000..ddf873f611a
>> --- /dev/null
>> +++ b/gas/testsuite/gas/riscv/zfbfmin.s
>> @@ -0,0 +1,4 @@
>> +target:
>> +	fcvt.s.bf16	fa0, fa1
>> +	fcvt.bf16.s	fa0, fa1
>> +	fcvt.bf16.s	fa0, fa1, rne
>> diff --git a/include/opcode/riscv-opc.h b/include/opcode/riscv-opc.h
>> index ae14e14d427..e33b81bf4fb 100644
>> --- a/include/opcode/riscv-opc.h
>> +++ b/include/opcode/riscv-opc.h
>> @@ -821,8 +821,12 @@
>>   #define MASK_FMIN_H 0xfe00707f
>>   #define MATCH_FMAX_H 0x2c001053
>>   #define MASK_FMAX_H 0xfe00707f
>> +#define MATCH_FCVT_BF16_S 0x44800053
>> +#define MASK_FCVT_BF16_S 0xfff0007f
>>   #define MATCH_FCVT_H_S 0x44000053
>>   #define MASK_FCVT_H_S 0xfff0007f
>> +#define MATCH_FCVT_S_BF16 0x40600053
>> +#define MASK_FCVT_S_BF16 0xfff0007f
>>   #define MATCH_FCVT_S_H 0x40200053
>>   #define MASK_FCVT_S_H 0xfff0007f
>>   #define MATCH_FSQRT_H 0x5c000053
>> @@ -3675,6 +3679,8 @@ DECLARE_INSN(fmin_d, MATCH_FMIN_D, MASK_FMIN_D)
>>   DECLARE_INSN(fmax_d, MATCH_FMAX_D, MASK_FMAX_D)
>>   DECLARE_INSN(fcvt_s_d, MATCH_FCVT_S_D, MASK_FCVT_S_D)
>>   DECLARE_INSN(fcvt_d_s, MATCH_FCVT_D_S, MASK_FCVT_D_S)
>> +DECLARE_INSN(fcvt_s_bf16, MATCH_FCVT_S_BF16 MASK_FCVT_S_BF16)
>> +DECLARE_INSN(fcvt_bf16_s, MATCH_FCVT_BF16_S, MASK_FCVT_BF16_S)
>>   DECLARE_INSN(fsqrt_d, MATCH_FSQRT_D, MASK_FSQRT_D)
>>   DECLARE_INSN(fadd_q, MATCH_FADD_Q, MASK_FADD_Q)
>>   DECLARE_INSN(fsub_q, MATCH_FSUB_Q, MASK_FSUB_Q)
>> diff --git a/include/opcode/riscv.h b/include/opcode/riscv.h
>> index 5f516a1026e..e10b2b890b5 100644
>> --- a/include/opcode/riscv.h
>> +++ b/include/opcode/riscv.h
>> @@ -446,6 +446,7 @@ enum riscv_insn_class
>>     INSN_CLASS_ZFHMIN_INX,
>>     INSN_CLASS_ZFHMIN_AND_D_INX,
>>     INSN_CLASS_ZFHMIN_AND_Q_INX,
>> +  INSN_CLASS_ZFBFMIN,
>>     INSN_CLASS_ZFA,
>>     INSN_CLASS_D_AND_ZFA,
>>     INSN_CLASS_Q_AND_ZFA,
>> diff --git a/opcodes/riscv-opc.c b/opcodes/riscv-opc.c
>> index 1ef4eaddf4d..f73d2c7a81d 100644
>> --- a/opcodes/riscv-opc.c
>> +++ b/opcodes/riscv-opc.c
>> @@ -764,6 +764,11 @@ const struct riscv_opcode riscv_opcodes[] =
>>   {"remw",      64, INSN_CLASS_M,     "d,s,t",     MATCH_REMW, MASK_REMW, match_opcode, 0 },
>>   {"remuw",     64, INSN_CLASS_M,     "d,s,t",     MATCH_REMUW, MASK_REMUW, match_opcode, 0 },
>>   
>> +/* BF16 floating-point instruction subset.  */
>> +{"fcvt.bf16.s",   0, INSN_CLASS_ZFBFMIN, "D,S",     MATCH_FCVT_BF16_S|MASK_RM, MASK_FCVT_BF16_S|MASK_RM, match_opcode, 0 },
>> +{"fcvt.bf16.s",   0, INSN_CLASS_ZFBFMIN, "D,S,m",   MATCH_FCVT_BF16_S, MASK_FCVT_BF16_S, match_opcode, 0 },
>> +{"fcvt.s.bf16",   0, INSN_CLASS_ZFBFMIN, "D,S",     MATCH_FCVT_S_BF16, MASK_FCVT_S_BF16|MASK_RM, match_opcode, 0 },
>> +
>>   /* Half-precision floating-point instruction subset.  */
>>   {"flh",        0, INSN_CLASS_ZFHMIN,   "D,o(s)",    MATCH_FLH, MASK_FLH, match_opcode, INSN_DREF|INSN_2_BYTE },
>>   {"flh",        0, INSN_CLASS_ZFHMIN,   "D,A,s",     0, (int) M_FLx, match_rs1_nonzero, INSN_MACRO },


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-06-05  0:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-06-04 11:09 [PATCH] RISC-V: Support Zfbfmin extension Jiawei
2024-06-04 16:52 ` Jan Beulich
2024-06-05  0:25   ` Jiawei

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).