Re: [PATCH V6] VECT: Support LEN_MASK_{LOAD,STORE} ifn && optabs

2023-06-19 Thread Jeff Law via Gcc-patches
On 6/19/23 00:56, Robin Dapp wrote: If the pattern is not allowed to fail, then what code enforces the bias argument's restrictions?  I don't see it in the generic expander code. I have no ideal since this is just copied from len_load/len_store which is s390 target dependent stuff. I have s

Re: [PATCH V6] VECT: Support LEN_MASK_{LOAD,STORE} ifn && optabs

2023-06-18 Thread Robin Dapp via Gcc-patches
>>>If the pattern is not allowed to fail, then what code enforces the bias >>>argument's restrictions?  I don't see it in the generic expander code. > > I have no ideal since this is just copied from len_load/len_store which is  > s390 target dependent stuff.  > > I have sent V7 patch with fixing

Re: Re: [PATCH V6] VECT: Support LEN_MASK_{LOAD,STORE} ifn && optabs

2023-06-17 Thread 钟居哲
uggestion. Thanks. juzhe.zh...@rivai.ai From: Jeff Law Date: 2023-06-17 22:19 To: juzhe.zhong; gcc-patches CC: richard.sandiford; rguenther; rdapp.gcc Subject: Re: [PATCH V6] VECT: Support LEN_MASK_{LOAD,STORE} ifn && optabs On 6/16/23 04:29, juzhe.zh...@rivai.ai wrote: > From: Ju

Re: [PATCH V6] VECT: Support LEN_MASK_{LOAD,STORE} ifn && optabs

2023-06-17 Thread Jeff Law via Gcc-patches
On 6/16/23 04:29, juzhe.zh...@rivai.ai wrote: From: Ju-Zhe Zhong This patch bootstrap pass on X86, ok for trunk ? Accoding to comments from Richi, split the first patch to add ifn && optabs of LEN_MASK_{LOAD,STORE} only, we don't apply them into vectorizer in this patch. And also add BIAS a

[PATCH V6] VECT: Support LEN_MASK_{LOAD,STORE} ifn && optabs

2023-06-16 Thread juzhe . zhong
From: Ju-Zhe Zhong This patch bootstrap pass on X86, ok for trunk ? Accoding to comments from Richi, split the first patch to add ifn && optabs of LEN_MASK_{LOAD,STORE} only, we don't apply them into vectorizer in this patch. And also add BIAS argument for possible s390's future use. The descri