Not sure since I saw MASK_STORE/LEN_STORE didn't compute size.
juzhe.zh...@rivai.ai
From: Jeff Law
Date: 2023-06-24 03:20
To: juzhe.zhong; gcc-patches
CC: rguenther; richard.sandiford
Subject: Re: [PATCH] SSA ALIAS: Apply LEN_MASK_{LOAD, STORE} into SSA alias
analysis
On 6/23/23 07:56, juzhe.zh...@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zh...@rivai.ai>
>
> gcc/ChangeLog:
>
> * tree-ssa-alias.cc (ref_maybe_used_by_call_p_1): Apply
> LEN_MASK_{LOAD,STORE}
>
> ---
> gcc/tree-ssa-alias.cc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/gcc/tree-ssa-alias.cc b/gcc/tree-ssa-alias.cc
> index e1bc04b82ba..92dc1bb9987 100644
> --- a/gcc/tree-ssa-alias.cc
> +++ b/gcc/tree-ssa-alias.cc
> @@ -2815,11 +2815,13 @@ ref_maybe_used_by_call_p_1 (gcall *call, ao_ref *ref,
> bool tbaa_p)
> case IFN_SCATTER_STORE:
> case IFN_MASK_SCATTER_STORE:
> case IFN_LEN_STORE:
> + case IFN_LEN_MASK_STORE:
> return false;
> case IFN_MASK_STORE_LANES:
> goto process_args;
> case IFN_MASK_LOAD:
> case IFN_LEN_LOAD:
> + case IFN_LEN_MASK_LOAD:
> case IFN_MASK_LOAD_LANES:
> {
> ao_ref rhs_ref;
Don't you need to adjust how you compute the size for the LEN_MASK_LOAD
case?
jeff
