Committed, thanks Jeff. Pan
-----Original Message----- From: Gcc-patches <gcc-patches-bounces+pan2.li=intel....@gcc.gnu.org> On Behalf Of Jeff Law via Gcc-patches Sent: Saturday, June 24, 2023 10:45 PM To: juzhe.zh...@rivai.ai; gcc-patches@gcc.gnu.org Cc: rguent...@suse.de; richard.sandif...@arm.com Subject: Re: [PATCH] GIMPLE_FOLD: Apply LEN_MASK_{LOAD, STORE} into GIMPLE_FOLD On 6/23/23 07:48, juzhe.zh...@rivai.ai wrote: > From: Ju-Zhe Zhong <juzhe.zh...@rivai.ai> > > Hi, since we are going to have LEN_MASK_{LOAD,STORE} into loopVectorizer. > > Currenly, > 1. we can fold MASK_{LOAD,STORE} into MEM when mask is all ones. > 2. we can fold LEN_{LOAD,STORE} into MEM when (len - bias) is VF. > > Now, I think it makes sense that we can support > > fold LEN_MASK_{LOAD,STORE} into MEM when both mask = all ones and (len - > bias) is VF. > > gcc/ChangeLog: > > * gimple-fold.cc (arith_overflowed_p): Apply LEN_MASK_{LOAD,STORE}. > (gimple_fold_partial_load_store_mem_ref): Ditto. > (gimple_fold_partial_store): Ditto. > (gimple_fold_call): Ditto. OK jeff