Re: Re: [PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STORE

2023-06-26 Thread 钟居哲
RVV didn't use IFN for intrinsics and I am not sure whether the refactoring is necessary or not. Thanks. juzhe.zh...@rivai.ai From: juzhe.zhong Date: 2023-06-26 21:04 To: Richard Biener CC: gcc-patches; richard.sandiford Subject: Re: [PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STOR

Re: Re: [PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STORE

2023-06-26 Thread Richard Biener via Gcc-patches
gt; > juzhe.zh...@rivai.ai > > From: Richard Biener > Date: 2023-06-26 20:16 > To: Ju-Zhe Zhong > CC: gcc-patches; richard.sandiford > Subject: Re: [PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STORE > On Mon, 26 Jun 2023, juzhe.zh...@rivai.ai wrote: > > > From: Ju-Zhe

Re: Re: [PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STORE

2023-06-26 Thread juzhe.zh...@rivai.ai
a)); return b; } } It seems that assertion will fail When nunits = [2,2] , len + bias = 3, for example. I may be wrong. Thanks. juzhe.zh...@rivai.ai From: Richard Biener Date: 2023-06-26 20:16 To: Ju-Zhe Zhong CC: gcc-patches; richard.sandiford Subject: Re: [PATCH V3] SCCVN: Add L

Re: [PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STORE

2023-06-26 Thread Richard Biener via Gcc-patches
On Mon, 26 Jun 2023, juzhe.zh...@rivai.ai wrote: > From: Ju-Zhe Zhong > > Hi, Richi. It seems that we use nunits which is len + bias to iterate then we > can > simplify the codes. > > Also, I fixed behavior of len_store, > > Before this patch: >(len - bias) * BITS_PER_UNIT > After this pa

[PATCH V3] SCCVN: Add LEN_MASK_STORE and fix LEN_STORE

2023-06-26 Thread juzhe . zhong
From: Ju-Zhe Zhong Hi, Richi. It seems that we use nunits which is len + bias to iterate then we can simplify the codes. Also, I fixed behavior of len_store, Before this patch: (len - bias) * BITS_PER_UNIT After this patch: (len + bias) * BITS_PER_UNIT gcc/ChangeLog: * tree-