RE: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-03-01 Thread Li, Pan2
Sure thing. Pan -Original Message- From: Vineet Gupta Sent: Saturday, March 2, 2024 3:00 AM To: Li, Pan2 ; Kito Cheng ; 钟居哲 Cc: gcc-patches ; Wang, Yanzhang ; rdapp.gcc ; Jeff Law Subject: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV Hi Pan, On 2/28/24 17

Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-03-01 Thread Vineet Gupta
; gcc-patches > ; Wang, Yanzhang ; > rdapp.gcc ; Jeff Law > *Subject:* Re: Re: [PATCH v3] RISC-V: Introduce gcc option > mrvv-vector-bits for RVV > >   > > Hmm, maybe only keep --param=riscv-autovec-preference=none and remove > other two if we think that might still useful

Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-29 Thread Robin Dapp
> I think it makes more sense to remove the whole > --param=riscv-autovec-preference since we should use > -fno-tree-vectorize instead of --param=riscv-autovec-preference=none > which is more reasonable compile option for users. > > --param is just a internal testing option that we added before,

RE: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread Li, Pan2
To: 钟居哲 Cc: Li, Pan2 ; gcc-patches ; Wang, Yanzhang ; rdapp.gcc ; Jeff Law Subject: Re: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV Hmm, maybe only keep --param=riscv-autovec-preference=none and remove other two if we think that might still useful? But anyway I have

Re: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread juzhe.zh...@rivai.ai
should remove them. juzhe.zh...@rivai.ai From: Kito Cheng Date: 2024-02-28 22:55 To: 钟居哲 CC: pan2.li; gcc-patches; yanzhang.wang; rdapp.gcc; Jeff Law Subject: Re: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV Hmm, maybe only keep --param=riscv-autovec-preference=none

Re: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread Kito Cheng
gcc > ; jeffreyalaw > *Subject:* Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits > for RVV > Take one more look, I think this option should work and integrate with > --param=riscv-autovec-preference= since they have similar jobs but > slightly different. > >

Re: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread 钟居哲
I think it makes more sense to remove --param=riscv-autovec-preference and add -mrvv-vector-bits juzhe.zh...@rivai.ai From: Kito Cheng Date: 2024-02-28 20:56 To: pan2.li CC: gcc-patches; juzhe.zhong; yanzhang.wang; rdapp.gcc; jeffreyalaw Subject: Re: [PATCH v3] RISC-V: Introduce gcc

RE: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread Li, Pan2
..@gmail.com Subject: Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV Take one more look, I think this option should work and integrate with --param=riscv-autovec-preference= since they have similar jobs but slightly different. We have 3 value for --param=riscv-autovec-prefe

Re: [PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread Kito Cheng
Take one more look, I think this option should work and integrate with --param=riscv-autovec-preference= since they have similar jobs but slightly different. We have 3 value for --param=riscv-autovec-preference=: none, scalable and fixed-vlmax -mrvv-vector-bits=scalable is work like

[PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread pan2 . li
From: Pan Li This patch would like to introduce one new gcc option for RVV. To appoint the bits size of one RVV vector register. Valid arguments to '-mrvv-vector-bits=' are: * scalable * zvl The scalable will pick up the zvl*b in the march as the minimal vlen. For example, the minimal vlen

[PATCH v3] RISC-V: Introduce gcc option mrvv-vector-bits for RVV

2024-02-28 Thread pan2 . li
From: Pan Li This patch would like to introduce one new gcc option for RVV. To appoint the bits size of one RVV vector register. Valid arguments to '-mrvv-vector-bits=' are: * scalable * zvl The scalable will pick up the zvl*b in the march as the minimal vlen. For example, the minimal vlen