On Thu, Jul 10, 2025 at 09:14:23AM +0200, Juergen Christ wrote:
> The default setting of s390 for the parameter min-vect-loop-bound was
> set to 2 to prevent certain epilogue loop vectorizations in the past.
> Reevaluation of this parameter shows that this setting now is not
> needed anymore and sometimes even harmful.  Remove the overwrite to
> align s390 with other backends.

Ok for mainline.

Thanks,
Stefan

> 
> Signed-off-by: Juergen Christ <jchr...@linux.ibm.com>
> 
> gcc/ChangeLog:
> 
>       * config/s390/s390.cc (s390_option_override_internal): Remove override.
> ---
>  gcc/config/s390/s390.cc | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/gcc/config/s390/s390.cc b/gcc/config/s390/s390.cc
> index de9c15c7bd42..737b176766a2 100644
> --- a/gcc/config/s390/s390.cc
> +++ b/gcc/config/s390/s390.cc
> @@ -16566,9 +16566,6 @@ s390_option_override_internal (struct gcc_options 
> *opts,
>    else
>      SET_OPTION_IF_UNSET (opts, opts_set, param_vect_partial_vector_usage, 0);
>  
> -  /* Do not vectorize loops with a low trip count for now.  */
> -  SET_OPTION_IF_UNSET (opts, opts_set, param_min_vect_loop_bound, 2);
> -
>    /* Set the default alignment.  */
>    s390_default_align (opts);
>  
> -- 
> 2.43.5
> 

Reply via email to