| Message ID | 644737e9d806d1c9f9e5770153780efe8fc363ae.1691991126.git.research_trasio@irq.a4lg.com |
|---|---|
| State | New |
| Headers | show |
| Series | RISC-V: Make "prefetch.i" built-in usable | expand |
On 8/13/23 23:32, Tsukasa OI via Gcc-patches wrote: > From: Tsukasa OI <research_trasio@irq.a4lg.com> > > This commit adds support for the 'Zfa' extension containing additional > floating point instructions, version 0.1 (stable and approved). > > gcc/ChangeLog: > > * common/config/riscv/riscv-common.cc > (riscv_implied_info): Add implication 'Zfa' -> 'F'. > (riscv_ext_version_table) Add support for the 'Zfa' extension. > (riscv_ext_flag_table) Set MASK_ZFA if 'Zfa' is available. > * config/riscv/riscv-opts.h (MASK_ZFA, TARGET_ZFA): New. So I think this and Jin Ma's most recently posted Zfa bits are almost functionally equivalent. The only notable difference in this patch is Jin's work puts Zfa into its own subextension rather than in the existing zf subextension. I think that was done in the v10 patch from Jin in response to the implies/depends comment from Kito. So I'm inclined to say that's the preferred approach. Given that's the only notable difference between this patch and Jin's patch, I'm going to consider 1/2 in this series superseded by Jin's work. jeff
diff --git a/gcc/common/config/riscv/riscv-common.cc b/gcc/common/config/riscv/riscv-common.cc index 21f83f26371f..01d68856bc40 100644 --- a/gcc/common/config/riscv/riscv-common.cc +++ b/gcc/common/config/riscv/riscv-common.cc @@ -121,6 +121,7 @@ static const riscv_implied_info_t riscv_implied_info[] = {"zvksg", "zvks"}, {"zvksg", "zvkg"}, + {"zfa", "f"}, {"zfh", "zfhmin"}, {"zfhmin", "f"}, {"zvfhmin", "zve32f"}, @@ -257,6 +258,7 @@ static const struct riscv_ext_version riscv_ext_version_table[] = {"zvl32768b", ISA_SPEC_CLASS_NONE, 1, 0}, {"zvl65536b", ISA_SPEC_CLASS_NONE, 1, 0}, + {"zfa", ISA_SPEC_CLASS_NONE, 0, 1}, {"zfh", ISA_SPEC_CLASS_NONE, 1, 0}, {"zfhmin", ISA_SPEC_CLASS_NONE, 1, 0}, {"zvfhmin", ISA_SPEC_CLASS_NONE, 1, 0}, @@ -1400,6 +1402,7 @@ static const riscv_ext_flag_table_t riscv_ext_flag_table[] = {"zvl32768b", &gcc_options::x_riscv_zvl_flags, MASK_ZVL32768B}, {"zvl65536b", &gcc_options::x_riscv_zvl_flags, MASK_ZVL65536B}, + {"zfa", &gcc_options::x_riscv_zf_subext, MASK_ZFA}, {"zfhmin", &gcc_options::x_riscv_zf_subext, MASK_ZFHMIN}, {"zfh", &gcc_options::x_riscv_zf_subext, MASK_ZFH}, {"zvfhmin", &gcc_options::x_riscv_zf_subext, MASK_ZVFHMIN}, diff --git a/gcc/config/riscv/riscv-opts.h b/gcc/config/riscv/riscv-opts.h index aeea805b3425..e31ec7c4074a 100644 --- a/gcc/config/riscv/riscv-opts.h +++ b/gcc/config/riscv/riscv-opts.h @@ -249,11 +249,13 @@ enum riscv_entity #define MASK_ZFH (1 << 1) #define MASK_ZVFHMIN (1 << 2) #define MASK_ZVFH (1 << 3) +#define MASK_ZFA (1 << 4) #define TARGET_ZFHMIN ((riscv_zf_subext & MASK_ZFHMIN) != 0) #define TARGET_ZFH ((riscv_zf_subext & MASK_ZFH) != 0) #define TARGET_ZVFHMIN ((riscv_zf_subext & MASK_ZVFHMIN) != 0) #define TARGET_ZVFH ((riscv_zf_subext & MASK_ZVFH) != 0) +#define TARGET_ZFA ((riscv_zf_subext & MASK_ZFA) != 0) #define MASK_ZMMUL (1 << 0) #define TARGET_ZMMUL ((riscv_zm_subext & MASK_ZMMUL) != 0)
From: Tsukasa OI <research_trasio@irq.a4lg.com> This commit adds support for the 'Zfa' extension containing additional floating point instructions, version 0.1 (stable and approved). gcc/ChangeLog: * common/config/riscv/riscv-common.cc (riscv_implied_info): Add implication 'Zfa' -> 'F'. (riscv_ext_version_table) Add support for the 'Zfa' extension. (riscv_ext_flag_table) Set MASK_ZFA if 'Zfa' is available. * config/riscv/riscv-opts.h (MASK_ZFA, TARGET_ZFA): New. --- gcc/common/config/riscv/riscv-common.cc | 3 +++ gcc/config/riscv/riscv-opts.h | 2 ++ 2 files changed, 5 insertions(+)