> 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: juzhe.zh...@rivai.ai <juzhe.zh...@rivai.ai> Sent: Friday, February 23, 2024 4:38 PM To: jeffreyalaw <jeffreya...@gmail.com>; kito.cheng <kito.ch...@gmail.com>; Li, Pan2 <pan2...@intel.com> Cc: gcc-patches <gcc-patches@gcc.gnu.org>; Wang, Yanzhang <yanzhang.w...@intel.com>; Robin Dapp <rdapp....@gmail.com>; palmer <pal...@rivosinc.com>; vineetg <vine...@rivosinc.com>; Patrick O'Neill <patr...@rivosinc.com>; Edwin Lu <e...@rivosinc.com> 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. ________________________________ juzhe.zh...@rivai.ai<mailto:juzhe.zh...@rivai.ai> From: Jeff Law<mailto:jeffreya...@gmail.com> Date: 2024-02-23 16:29 To: Kito Cheng<mailto:kito.ch...@gmail.com>; pan2.li<mailto:pan2...@intel.com> CC: gcc-patches<mailto:gcc-patches@gcc.gnu.org>; juzhe.zhong<mailto:juzhe.zh...@rivai.ai>; yanzhang.wang<mailto:yanzhang.w...@intel.com> 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
