Re: [PATCH v2] vect: Check that vector factor is a compile-time constant

[Richard Biener <richard.guenther@gmail.com>]

On Thu, Mar 9, 2023 at 8:57 AM Michael Collison <collison@rivosinc.com> wrote:

OK.

Thanks,
Richard.

> 2023-03-05  Michael Collison  <collison@rivosinc.com>
>
>         * tree-vect-loop-manip.cc (vect_do_peeling): Use
>         result of constant_lower_bound instead of vf in case
>         vf is not a compile time constant.
> ---
>  gcc/tree-vect-loop-manip.cc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/tree-vect-loop-manip.cc b/gcc/tree-vect-loop-manip.cc
> index d88edafa018..f60fa50e8f4 100644
> --- a/gcc/tree-vect-loop-manip.cc
> +++ b/gcc/tree-vect-loop-manip.cc
> @@ -2921,7 +2921,7 @@ vect_do_peeling (loop_vec_info loop_vinfo, tree niters, tree nitersm1,
>        if (new_var_p)
>         {
>           value_range vr (type,
> -                         wi::to_wide (build_int_cst (type, vf)),
> +                         wi::to_wide (build_int_cst (type, lowest_vf)),
>                           wi::to_wide (TYPE_MAX_VALUE (type)));
>           set_range_info (niters, vr);
>         }
> --
> 2.34.1
>