On 8/13/23 23:32, Tsukasa OI via Gcc-patches wrote:
From: Tsukasa OI <research_tra...@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