diff mbox series

Disable --param vect-partial-vector-usage by default on x86

Message ID 9823nrr8-78or-p4q2-r1o2-3s8r53rs169@fhfr.qr
State New
Headers show
Series Disable --param vect-partial-vector-usage by default on x86 | expand

Commit Message

Richard Biener July 15, 2021, 12:32 p.m. UTC
The following defaults --param vect-partial-vector-usage to zero
for x86_64 matching existing behavior where support for this
is not present.

OK for trunk?

Thanks,
Richard/

2021-07-15  Richard Biener  <rguenther@suse.de>

	* config/i386/i386-options.c (ix86_option_override_internal): Set
	param_vect_partial_vector_usage to zero if not set.
---
 gcc/config/i386/i386-options.c | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Hongtao Liu July 16, 2021, 1:53 a.m. UTC | #1
On Thu, Jul 15, 2021 at 8:33 PM Richard Biener <rguenther@suse.de> wrote:
>
> The following defaults --param vect-partial-vector-usage to zero
> for x86_64 matching existing behavior where support for this
> is not present.
>
> OK for trunk?
>
Ok.
> Thanks,
> Richard/
>
> 2021-07-15  Richard Biener  <rguenther@suse.de>
>
>         * config/i386/i386-options.c (ix86_option_override_internal): Set
>         param_vect_partial_vector_usage to zero if not set.
> ---
>  gcc/config/i386/i386-options.c | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/gcc/config/i386/i386-options.c b/gcc/config/i386/i386-options.c
> index 7cba655595e..3416a4f1752 100644
> --- a/gcc/config/i386/i386-options.c
> +++ b/gcc/config/i386/i386-options.c
> @@ -2834,6 +2834,11 @@ ix86_option_override_internal (bool main_args_p,
>
>    SET_OPTION_IF_UNSET (opts, opts_set, param_ira_consider_dup_in_all_alts, 0);
>
> +  /* Fully masking the main or the epilogue vectorized loop is not
> +     profitable generally so leave it disabled until we get more
> +     fine grained control & costing.  */
> +  SET_OPTION_IF_UNSET (opts, opts_set, param_vect_partial_vector_usage, 0);
> +
>    return true;
>  }
>
> --
> 2.26.2
diff mbox series

Patch

diff --git a/gcc/config/i386/i386-options.c b/gcc/config/i386/i386-options.c
index 7cba655595e..3416a4f1752 100644
--- a/gcc/config/i386/i386-options.c
+++ b/gcc/config/i386/i386-options.c
@@ -2834,6 +2834,11 @@  ix86_option_override_internal (bool main_args_p,
 
   SET_OPTION_IF_UNSET (opts, opts_set, param_ira_consider_dup_in_all_alts, 0);
 
+  /* Fully masking the main or the epilogue vectorized loop is not
+     profitable generally so leave it disabled until we get more
+     fine grained control & costing.  */
+  SET_OPTION_IF_UNSET (opts, opts_set, param_vect_partial_vector_usage, 0);
+
   return true;
 }