> I would prefer to only keep zvl and scalable or zvl only, since I > don't see too much value in specifying a value which different from
> zvl*b, that's a legacy option used before zvl*b option was introduced, > and the reason to add that is that could used for compatible with > clang/LLVM for riscv_rvv_vector_bits attribute I think? Yes, exactly to be compatible with clang/llvm. Just take zvl is good enough IMO, and update in v2 once we have alignment. > And if we want this (I'm not sure), it really feels like it ought to defer to > gcc-15. > But I'd like to CC more RISC-V GCC folks to see the votes. > If most of the people don't want this in GCC-14 and defer it to GCC-15, I > won't insist on it. Sure, let’s wait for a while. Pan From: [email protected] <[email protected]> Sent: Friday, February 23, 2024 4:38 PM To: jeffreyalaw <[email protected]>; kito.cheng <[email protected]>; Li, Pan2 <[email protected]> Cc: gcc-patches <[email protected]>; Wang, Yanzhang <[email protected]>; Robin Dapp <[email protected]>; palmer <[email protected]>; vineetg <[email protected]>; Patrick O'Neill <[email protected]>; Edwin Lu <[email protected]> Subject: Re: Re: [PATCH v1] RISC-V: Introduce gcc option mrvv-vector-bits for RVV I personally think it's better to has VLS compile option and attribute in GCC-14. Since there are many people porting different libraury (eigen/highway/xnnpack/openBLAS,...) with VLS feature, they test them with Clang. If we don't support it, we will end up with Clang can compile those lib but GCC-14 can't which will make RISC-V folks think GCC is still pretty far behind than Clang. Besides, VLS compile option and attribute are pretty safe codes, I would surprise that it will cause issues on current RVV support. So, +1 from my side to support VLS compile option and attribute on GCC-14. But I'd like to CC more RISC-V GCC folks to see the votes. If most of the people don't want this in GCC-14 and defer it to GCC-15, I won't insist on it. Thanks. ________________________________ [email protected]<mailto:[email protected]> From: Jeff Law<mailto:[email protected]> Date: 2024-02-23 16:29 To: Kito Cheng<mailto:[email protected]>; pan2.li<mailto:[email protected]> CC: gcc-patches<mailto:[email protected]>; juzhe.zhong<mailto:[email protected]>; yanzhang.wang<mailto:[email protected]> Subject: Re: [PATCH v1] RISC-V: Introduce gcc option mrvv-vector-bits for RVV On 2/23/24 01:22, Kito Cheng wrote: > I would prefer to only keep zvl and scalable or zvl only, since I > don't see too much value in specifying a value which different from > zvl*b, that's a legacy option used before zvl*b option was introduced, > and the reason to add that is that could used for compatible with > clang/LLVM for riscv_rvv_vector_bits attribute I think? And if we want this (I'm not sure), it really feels like it ought to defer to gcc-15. jeff
