Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-10-14 Thread Richard Biener via Gcc-patches
On Thu, Oct 14, 2021 at 2:29 PM Aldy Hernandez wrote: > > On Mon, Sep 27, 2021 at 7:29 PM Richard Biener > wrote: > > > > On September 27, 2021 6:07:40 PM GMT+02:00, Aldy Hernandez via Gcc-patches > > wrote: > > > > > > > > >On 9/27/21 5:27 PM, Aldy Hernandez wrote: > > >> > > >> > > >> On 9/27

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-10-14 Thread Aldy Hernandez via Gcc-patches
On Mon, Sep 27, 2021 at 7:29 PM Richard Biener wrote: > > On September 27, 2021 6:07:40 PM GMT+02:00, Aldy Hernandez via Gcc-patches > wrote: > > > > > >On 9/27/21 5:27 PM, Aldy Hernandez wrote: > >> > >> > >> On 9/27/21 5:01 PM, Jeff Law wrote: > >>> > >>> > >>> On 9/24/2021 9:46 AM, Aldy Herna

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-10-01 Thread Aldy Hernandez via Gcc-patches
On 10/1/21 12:55 PM, Gerald Pfeifer wrote: On Fri, 24 Sep 2021, Aldy Hernandez via Gcc-patches wrote: This patch implements the new hybrid forward threader and replaces the embedded VRP threader with it. I'm not sure this is the right of the patches to follow-up around this, but between Jef

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-10-01 Thread Gerald Pfeifer
On Fri, 24 Sep 2021, Aldy Hernandez via Gcc-patches wrote: > This patch implements the new hybrid forward threader and replaces the > embedded VRP threader with it. I'm not sure this is the right of the patches to follow-up around this, but between Jeff writing "Note we've got massive failures

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-29 Thread Jeff Law via Gcc-patches
On 9/29/2021 3:20 AM, Bernhard Reutner-Fischer wrote: On Wed, 29 Sep 2021 10:10:00 +0200 Aldy Hernandez wrote: Jeff has requested we slow down changes in the threading space while we chased down regressions. Sure. Take your time. That being said, thank you for your suggestion. I am putti

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-29 Thread Bernhard Reutner-Fischer via Gcc-patches
On Wed, 29 Sep 2021 10:10:00 +0200 Aldy Hernandez wrote: > Jeff has requested we slow down changes in the threading space while > we chased down regressions. Sure. Take your time. > > That being said, thank you for your suggestion. I am putting the > attached patch in my queue for testing. LG

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-27 Thread Richard Biener via Gcc-patches
On September 27, 2021 6:07:40 PM GMT+02:00, Aldy Hernandez via Gcc-patches wrote: > > >On 9/27/21 5:27 PM, Aldy Hernandez wrote: >> >> >> On 9/27/21 5:01 PM, Jeff Law wrote: >>> >>> >>> On 9/24/2021 9:46 AM, Aldy Hernandez wrote: > >>> And the big question, is the pass running after VRP2 doing

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-27 Thread Aldy Hernandez via Gcc-patches
On 9/27/21 5:27 PM, Aldy Hernandez wrote: On 9/27/21 5:01 PM, Jeff Law wrote: On 9/24/2021 9:46 AM, Aldy Hernandez wrote: And the big question, is the pass running after VRP2 doing anything particularly useful?  Do we want to try and kill it now, or later? Interesting question.  Perh

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-27 Thread Aldy Hernandez via Gcc-patches
On 9/27/21 5:01 PM, Jeff Law wrote: On 9/24/2021 9:46 AM, Aldy Hernandez wrote: This patch implements the new hybrid forward threader and replaces the embedded VRP threader with it. But most importantly, it pulls it out of the VRP pass as we no longer need the VRP data or ASSERT_EXPRs. Y

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-27 Thread Jeff Law via Gcc-patches
On 9/24/2021 9:46 AM, Aldy Hernandez wrote: This patch implements the new hybrid forward threader and replaces the embedded VRP threader with it. But most importantly, it pulls it out of the VRP pass as we no longer need the VRP data or ASSERT_EXPRs. With all the pieces that have gone in,

Re: [PATCH] Replace VRP threader with a hybrid forward threader.

2021-09-25 Thread Bernhard Reutner-Fischer via Gcc-patches
On Fri, 24 Sep 2021 17:46:53 +0200 Aldy Hernandez via Gcc-patches wrote: > p.s. "Did I say 5 weeks? My bad, I meant 5 months." heh. units (.oO~"xkcd.com/1047/") > +static unsigned int > +execute_vrp_threader (function *fun) > +{ > + hybrid_threader threader; > + threader.thread_jumps (fun);