Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-10-08 Thread Martin Sebor via Gcc-patches
On 10/8/21 4:49 AM, Aldy Hernandez via Gcc-patches wrote: On Thu, Sep 23, 2021 at 8:32 AM Richard Biener via Gcc-patches wrote: On Thu, 23 Sep 2021, Hongtao Liu wrote: On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: On 9/21/21

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-10-08 Thread Aldy Hernandez via Gcc-patches
On Thu, Sep 23, 2021 at 8:32 AM Richard Biener via Gcc-patches wrote: > > On Thu, 23 Sep 2021, Hongtao Liu wrote: > > > On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: > > > > > > On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: > > > > > > > > On 9/21/21 7:38 PM, Hongtao Liu wrote: > >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-10-07 Thread Hongtao Liu via Gcc-patches
On Thu, Oct 7, 2021 at 11:38 PM H.J. Lu via Gcc-patches wrote: > > On Thu, Oct 7, 2021 at 8:35 AM Martin Liška wrote: > > > > Hello. > > > > The patch is approved, are you planning committing the changes? Committed. > > > > Thanks, > > Martin > > Hongtao is on holiday. He will be back later

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-10-07 Thread H.J. Lu via Gcc-patches
On Thu, Oct 7, 2021 at 8:35 AM Martin Liška wrote: > > Hello. > > The patch is approved, are you planning committing the changes? > > Thanks, > Martin Hongtao is on holiday. He will be back later today. -- H.J.

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-10-07 Thread Martin Liška
Hello. The patch is approved, are you planning committing the changes? Thanks, Martin

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-28 Thread Richard Biener via Gcc-patches
On Sun, 26 Sep 2021, liuhongt wrote: > Hi: > > Please don't add the -fno- option to the warning tests.  As I said, > > I would prefer to either suppress the vectorization for the failing > > cases by tweaking the test code or xfail them.  That way future > > regressions won't be masked by the

[PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-25 Thread liuhongt via Gcc-patches
Hi: > Please don't add the -fno- option to the warning tests.  As I said, > I would prefer to either suppress the vectorization for the failing > cases by tweaking the test code or xfail them.  That way future > regressions won't be masked by the option.  Once we've moved > the warning to a more

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-24 Thread Martin Sebor via Gcc-patches
On 9/23/21 9:32 PM, Hongtao Liu wrote: On Thu, Sep 23, 2021 at 11:18 PM Martin Sebor wrote: On 9/23/21 12:30 AM, Richard Biener wrote: On Thu, 23 Sep 2021, Hongtao Liu wrote: On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: On

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-23 Thread Hongtao Liu via Gcc-patches
On Thu, Sep 23, 2021 at 11:18 PM Martin Sebor wrote: > > On 9/23/21 12:30 AM, Richard Biener wrote: > > On Thu, 23 Sep 2021, Hongtao Liu wrote: > > > >> On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: > >>> > >>> On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: > > On 9/21/21

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-23 Thread Martin Sebor via Gcc-patches
On 9/23/21 12:30 AM, Richard Biener wrote: On Thu, 23 Sep 2021, Hongtao Liu wrote: On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: On 9/21/21 7:38 PM, Hongtao Liu wrote: On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: ...

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-23 Thread Richard Biener via Gcc-patches
On Thu, 23 Sep 2021, Hongtao Liu wrote: > On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: > > > > On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: > > > > > > On 9/21/21 7:38 PM, Hongtao Liu wrote: > > > > On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: > > > ... > > > > diff

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-22 Thread Hongtao Liu via Gcc-patches
On Thu, Sep 23, 2021 at 9:48 AM Hongtao Liu wrote: > > On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: > > > > On 9/21/21 7:38 PM, Hongtao Liu wrote: > > > On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: > > ... > > > diff --git a/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-22 Thread Hongtao Liu via Gcc-patches
On Wed, Sep 22, 2021 at 10:21 PM Martin Sebor wrote: > > On 9/21/21 7:38 PM, Hongtao Liu wrote: > > On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: > ... > > diff --git a/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c > > b/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-22 Thread Martin Sebor via Gcc-patches
On 9/22/21 8:21 AM, Martin Sebor wrote: On 9/21/21 7:38 PM, Hongtao Liu wrote: On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: ... diff --git a/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c b/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c index 1d79930cd58..9351f7e7a1a 100644 ---

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-22 Thread Martin Sebor via Gcc-patches
On 9/21/21 7:38 PM, Hongtao Liu wrote: On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: ... diff --git a/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c b/gcc/testsuite/c-c++-common/Wstringop-overflow-2.c index 1d79930cd58..9351f7e7a1a 100644 ---

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-21 Thread Hongtao Liu via Gcc-patches
On Mon, Sep 20, 2021 at 4:13 AM Martin Sebor wrote: > > On 9/16/21 3:03 AM, Hongtao Liu via Gcc-patches wrote: > > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > > wrote: > >> > >> On Thu, 16 Sep 2021, liuhongt wrote: > >> > >>> Ping > >>> rebased on latest trunk. > >>> > >>>

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-19 Thread Martin Sebor via Gcc-patches
On 9/16/21 3:03 AM, Hongtao Liu via Gcc-patches wrote: On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches wrote: On Thu, 16 Sep 2021, liuhongt wrote: Ping rebased on latest trunk. gcc/ChangeLog: * common.opt (ftree-vectorize): Add Var(flag_tree_vectorize). *

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-17 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 17, 2021 at 3:47 PM Richard Biener wrote: > > On Fri, 17 Sep 2021, Hongtao Liu wrote: > > > On Thu, Sep 16, 2021 at 8:31 PM Richard Biener wrote: > > > > > > On Thu, 16 Sep 2021, Hongtao Liu wrote: > > > > > > > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > > > >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-17 Thread Richard Biener via Gcc-patches
On Fri, 17 Sep 2021, Hongtao Liu wrote: > On Thu, Sep 16, 2021 at 8:31 PM Richard Biener wrote: > > > > On Thu, 16 Sep 2021, Hongtao Liu wrote: > > > > > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > > > wrote: > > > > > > > > On Thu, 16 Sep 2021, liuhongt wrote: > > > > > >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-16 Thread Hongtao Liu via Gcc-patches
On Thu, Sep 16, 2021 at 8:31 PM Richard Biener wrote: > > On Thu, 16 Sep 2021, Hongtao Liu wrote: > > > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > > wrote: > > > > > > On Thu, 16 Sep 2021, liuhongt wrote: > > > > > > > Ping > > > > rebased on latest trunk. > > > > > > > >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, Hongtao Liu wrote: > On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches > wrote: > > > > On Thu, 16 Sep 2021, liuhongt wrote: > > > > > Ping > > > rebased on latest trunk. > > > > > > gcc/ChangeLog: > > > > > > * common.opt (ftree-vectorize): Add

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-16 Thread Hongtao Liu via Gcc-patches
On Thu, Sep 16, 2021 at 4:23 PM Richard Biener via Gcc-patches wrote: > > On Thu, 16 Sep 2021, liuhongt wrote: > > > Ping > > rebased on latest trunk. > > > > gcc/ChangeLog: > > > > * common.opt (ftree-vectorize): Add Var(flag_tree_vectorize). > > * doc/invoke.texi (Options That

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-16 Thread Richard Biener via Gcc-patches
On Thu, 16 Sep 2021, liuhongt wrote: > Ping > rebased on latest trunk. > > gcc/ChangeLog: > > * common.opt (ftree-vectorize): Add Var(flag_tree_vectorize). > * doc/invoke.texi (Options That Control Optimization): Update > documents. > * opts.c (default_options_table):

[PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-15 Thread liuhongt via Gcc-patches
Ping rebased on latest trunk. gcc/ChangeLog: * common.opt (ftree-vectorize): Add Var(flag_tree_vectorize). * doc/invoke.texi (Options That Control Optimization): Update documents. * opts.c (default_options_table): Enable auto-vectorization at O2 with

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Hongtao Liu via Gcc-patches
On Tue, Sep 7, 2021 at 12:37 AM Joseph Myers wrote: > > On Mon, 6 Sep 2021, liuhongt via Gcc-patches wrote: > > > Hi: > > As discussed in [1], most of (currently unopposed) targets want > > auto-vectorization at O2, and IMHO now would be a good time to enable O2 > > vectorization for GCC trunk,

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Joseph Myers
On Mon, 6 Sep 2021, liuhongt via Gcc-patches wrote: > Hi: > As discussed in [1], most of (currently unopposed) targets want > auto-vectorization at O2, and IMHO now would be a good time to enable O2 > vectorization for GCC trunk, so it would leave enough time to expose > related issues and fix

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Richard Biener via Gcc-patches
On Mon, Sep 6, 2021 at 2:30 PM Jakub Jelinek wrote: > > On Mon, Sep 06, 2021 at 02:18:59PM +0200, Richard Biener wrote: > > On Mon, Sep 6, 2021 at 1:15 PM Jakub Jelinek wrote: > > > > > > On Mon, Sep 06, 2021 at 07:15:41PM +0800, Hongtao Liu wrote: > > > > > So what about finish_options then? >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Jakub Jelinek via Gcc-patches
On Mon, Sep 06, 2021 at 02:18:59PM +0200, Richard Biener wrote: > On Mon, Sep 6, 2021 at 1:15 PM Jakub Jelinek wrote: > > > > On Mon, Sep 06, 2021 at 07:15:41PM +0800, Hongtao Liu wrote: > > > > So what about finish_options then? > > > > default_options_optimization has only a single caller that

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Richard Biener via Gcc-patches
On Mon, Sep 6, 2021 at 1:15 PM Jakub Jelinek wrote: > > On Mon, Sep 06, 2021 at 07:15:41PM +0800, Hongtao Liu wrote: > > > So what about finish_options then? > > > default_options_optimization has only a single caller that then calls > > > read_cmdline_options and then finish_options. > > in

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Jakub Jelinek via Gcc-patches
On Mon, Sep 06, 2021 at 07:15:41PM +0800, Hongtao Liu wrote: > > So what about finish_options then? > > default_options_optimization has only a single caller that then calls > > read_cmdline_options and then finish_options. > in finish_options > (gdb) p opts_set->x_flag_tree_loop_vectorize > $37 =

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Hongtao Liu via Gcc-patches
On Mon, Sep 6, 2021 at 7:01 PM Jakub Jelinek wrote: > > On Mon, Sep 06, 2021 at 06:58:37PM +0800, Hongtao Liu wrote: > > > process_options would mean it affects only the command line and not > > > __attribute__((optimize ("O2", "ftree-vectorize"))) > > > etc. > > > So, shouldn't it be instead

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Richard Biener via Gcc-patches
On Mon, Sep 6, 2021 at 11:41 AM Jakub Jelinek wrote: > > On Mon, Sep 06, 2021 at 11:18:47AM +0200, Richard Biener wrote: > > On Mon, Sep 6, 2021 at 10:47 AM liuhongt via Gcc-patches > > wrote: > > > > > > Hi: > > > As discussed in [1], most of (currently unopposed) targets want > > >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Jakub Jelinek via Gcc-patches
On Mon, Sep 06, 2021 at 06:58:37PM +0800, Hongtao Liu wrote: > > process_options would mean it affects only the command line and not > > __attribute__((optimize ("O2", "ftree-vectorize"))) > > etc. > > So, shouldn't it be instead done in default_options_optimization, somewhere > It seems

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Hongtao Liu via Gcc-patches
On Mon, Sep 6, 2021 at 5:42 PM Jakub Jelinek via Gcc-patches wrote: > > On Mon, Sep 06, 2021 at 11:18:47AM +0200, Richard Biener wrote: > > On Mon, Sep 6, 2021 at 10:47 AM liuhongt via Gcc-patches > > wrote: > > > > > > Hi: > > > As discussed in [1], most of (currently unopposed) targets want

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Jakub Jelinek via Gcc-patches
On Mon, Sep 06, 2021 at 11:18:47AM +0200, Richard Biener wrote: > On Mon, Sep 6, 2021 at 10:47 AM liuhongt via Gcc-patches > wrote: > > > > Hi: > > As discussed in [1], most of (currently unopposed) targets want > > auto-vectorization at O2, and IMHO now would be a good time to enable O2 > >

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Hongtao Liu via Gcc-patches
On Mon, Sep 6, 2021 at 5:19 PM Richard Biener via Gcc-patches wrote: > > On Mon, Sep 6, 2021 at 10:47 AM liuhongt via Gcc-patches > wrote: > > > > Hi: > > As discussed in [1], most of (currently unopposed) targets want > > auto-vectorization at O2, and IMHO now would be a good time to enable

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Richard Biener via Gcc-patches
On Mon, Sep 6, 2021 at 10:47 AM liuhongt via Gcc-patches wrote: > > Hi: > As discussed in [1], most of (currently unopposed) targets want > auto-vectorization at O2, and IMHO now would be a good time to enable O2 > vectorization for GCC trunk, so it would leave enough time to expose > related

Re: [PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread Hongtao Liu via Gcc-patches
On Mon, Sep 6, 2021 at 4:46 PM liuhongt via Gcc-patches wrote: > > Hi: > As discussed in [1], most of (currently unopposed) targets want > auto-vectorization at O2, and IMHO now would be a good time to enable O2 > vectorization for GCC trunk, so it would leave enough time to expose > related

[PATCH] Enable auto-vectorization at O2 with very-cheap cost model.

2021-09-06 Thread liuhongt via Gcc-patches
Hi: As discussed in [1], most of (currently unopposed) targets want auto-vectorization at O2, and IMHO now would be a good time to enable O2 vectorization for GCC trunk, so it would leave enough time to expose related issues and fix them. Bootstrapped and regtested on x86_64-linux-gnu{-m32,}