"Kewen.Lin" <li...@linux.ibm.com> writes:
> Hi,
>
> Commit r14-2267-gb8806f6ffbe72e adjusts the arguments order
> of LEN_STORE from {len,vector,bias} to {len,bias,vector},
> in order to make them consistent with LEN_MASK_STORE and
> MASK_STORE. But it missed to update the related handlings
> in tree-ssa-sccvn.cc, it caused the failure shown in PR
> 110744. This patch is to fix the related handlings with
> the correct index.
>
> Bootstrapped and regress-tested on x86_64-redhat-linux,
> powerpc64-linux-gnu P8/P9 and powerpc64le-linux-gnu P9/P10.
>
> Is it ok for trunk?
>
> BR,
> Kewen
> -----
> PR tree-optimization/110744
>
> gcc/ChangeLog:
>
> * tree-ssa-sccvn.cc (vn_reference_lookup_3): Correct the index of bias
> operand for ifn IFN_LEN_STORE.
OK, thanks.
Richard
> ---
> gcc/tree-ssa-sccvn.cc | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/gcc/tree-ssa-sccvn.cc b/gcc/tree-ssa-sccvn.cc
> index 11061a374a2..c0b3ec420c5 100644
> --- a/gcc/tree-ssa-sccvn.cc
> +++ b/gcc/tree-ssa-sccvn.cc
> @@ -3299,11 +3299,14 @@ vn_reference_lookup_3 (ao_ref *ref, tree vuse, void
> *data_,
> return (void *)-1;
> break;
> case IFN_LEN_STORE:
> - len = gimple_call_arg (call, 2);
> - bias = gimple_call_arg (call, 4);
> - if (!tree_fits_uhwi_p (len) || !tree_fits_shwi_p (bias))
> - return (void *)-1;
> - break;
> + {
> + int len_index = internal_fn_len_index (fn);
> + len = gimple_call_arg (call, len_index);
> + bias = gimple_call_arg (call, len_index + 1);
> + if (!tree_fits_uhwi_p (len) || !tree_fits_shwi_p (bias))
> + return (void *) -1;
> + break;
> + }
> default:
> return (void *)-1;
> }
> --
> 2.39.3
