RE: [PATCH] GIMPLE_FOLD: Apply LEN_MASK_{LOAD, STORE} into GIMPLE_FOLD

["Li, Pan2" <pan2.li@intel.com>]

Committed, thanks Jeff.

Pan

-----Original Message-----
From: Gcc-patches <gcc-patches-bounces+pan2.li=intel.com@gcc.gnu.org> On Behalf Of Jeff Law via Gcc-patches
Sent: Saturday, June 24, 2023 10:45 PM
To: juzhe.zhong@rivai.ai; gcc-patches@gcc.gnu.org
Cc: rguenther@suse.de; richard.sandiford@arm.com
Subject: Re: [PATCH] GIMPLE_FOLD: Apply LEN_MASK_{LOAD, STORE} into GIMPLE_FOLD



On 6/23/23 07:48, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@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