Actually, I don't check and test HF carefully since I disable them. Kito ask me to disable all HF modes since zvfhmin is no ratified and GCC doesn't allow any un-ratified ISA. You can see vector-iterator.md that all RVV modes supported including QI HI SI DI SF DF excluding HF and BF.
juzhe.zh...@rivai.ai From: Jeff Law Date: 2022-12-17 09:48 To: juzhe.zhong; gcc-patches CC: kito.cheng; palmer Subject: Re: [PATCH] RISC-V: Fix RVV machine mode attribute configuration On 12/14/22 00:01, juzhe.zh...@rivai.ai wrote: > From: Ju-Zhe Zhong <juzhe.zh...@rivai.ai> > > The attribute configuration of each machine mode are support in the previous > patch. > I noticed some of them are not correct during VSETVL PASS testsing. > Correct them in the single patch now. > > gcc/ChangeLog: > > * config/riscv/riscv-vector-switch.def (ENTRY): Correct attributes. > > @@ -121,7 +121,7 @@ ENTRY (VNx2HI, true, LMUL_1, 16, LMUL_F2, 32) > ENTRY (VNx1HI, true, LMUL_F2, 32, LMUL_F4, 64) > > /* TODO:Disable all FP16 vector, enable them when 'zvfh' is supported. */ > -ENTRY (VNx32HF, false, LMUL_8, 2, LMUL_RESERVED, 0) > +ENTRY (VNx32HF, false, LMUL_RESERVED, 0, LMUL_8, 2) Is there any value in making VNx32HF dependent on TARGET_MIN_VLEN > 32 like we're doing for VNx32HI? In the past I've found it useful to have HI, HF, BF behave identically as much as possible. You call. The patch is OK either way. jeff
