Re: [patch] jump threading multiple paths that start from the same BB

2018-07-10 Thread Aldy Hernandez
On 07/09/2018 03:56 PM, Jeff Law wrote: On 07/09/2018 01:19 AM, Aldy Hernandez wrote: I'd like decisions about how to expand branches deferred until rtl expansion.  Kai was poking at this in the past but never really got any traction. For the record, the problem in this testcase is that sw

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-09 Thread Jeff Law
On 07/09/2018 01:19 AM, Aldy Hernandez wrote: >> >> I'd like decisions about how to expand branches deferred until rtl >> expansion.  Kai was poking at this in the past but never really got any >> traction. > > For the record, the problem in this testcase is that switch lowering is > riddled with

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-09 Thread Aldy Hernandez
On 07/09/2018 04:29 AM, Richard Biener wrote: On Mon, Jul 9, 2018 at 9:19 AM Aldy Hernandez wrote: On 07/05/2018 03:27 PM, Jeff Law wrote: On 07/04/2018 02:12 AM, Aldy Hernandez wrote: Many tests should turn into gimple IL tests. Yeah, though for tests like the threading ones, they'

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-09 Thread Richard Biener
On Mon, Jul 9, 2018 at 9:19 AM Aldy Hernandez wrote: > > > > On 07/05/2018 03:27 PM, Jeff Law wrote: > > On 07/04/2018 02:12 AM, Aldy Hernandez wrote: > >> > >> > >> On 07/03/2018 08:16 PM, Jeff Law wrote: > >>> On 07/03/2018 03:31 AM, Aldy Hernandez wrote: > On 07/02/2018 07:08 AM, Christoph

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-09 Thread Aldy Hernandez
On 07/05/2018 03:27 PM, Jeff Law wrote: On 07/04/2018 02:12 AM, Aldy Hernandez wrote: On 07/03/2018 08:16 PM, Jeff Law wrote: On 07/03/2018 03:31 AM, Aldy Hernandez wrote: On 07/02/2018 07:08 AM, Christophe Lyon wrote: On 11/07/2017 10:33 AM, Aldy Hernandez wrote: While poking around i

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-05 Thread Jeff Law
On 07/04/2018 02:12 AM, Aldy Hernandez wrote: > > > On 07/03/2018 08:16 PM, Jeff Law wrote: >> On 07/03/2018 03:31 AM, Aldy Hernandez wrote: >>> On 07/02/2018 07:08 AM, Christophe Lyon wrote: >>> >> On 11/07/2017 10:33 AM, Aldy Hernandez wrote: >>> While poking around in the backwards thr

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-04 Thread Richard Biener
On Wed, Jul 4, 2018 at 10:12 AM Aldy Hernandez wrote: > > > > On 07/03/2018 08:16 PM, Jeff Law wrote: > > On 07/03/2018 03:31 AM, Aldy Hernandez wrote: > >> On 07/02/2018 07:08 AM, Christophe Lyon wrote: > >> > > On 11/07/2017 10:33 AM, Aldy Hernandez wrote: > >> While poking around in the

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-04 Thread Aldy Hernandez
On 07/03/2018 08:16 PM, Jeff Law wrote: On 07/03/2018 03:31 AM, Aldy Hernandez wrote: On 07/02/2018 07:08 AM, Christophe Lyon wrote: On 11/07/2017 10:33 AM, Aldy Hernandez wrote: While poking around in the backwards threader I noticed that we bail if we have already seen a starting BB.

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-03 Thread Jeff Law
On 07/03/2018 03:31 AM, Aldy Hernandez wrote: > On 07/02/2018 07:08 AM, Christophe Lyon wrote: > On 11/07/2017 10:33 AM, Aldy Hernandez wrote: > While poking around in the backwards threader I noticed that we bail if > > we have already seen a starting BB. > > /* D

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-03 Thread Aldy Hernandez
On 07/02/2018 07:08 AM, Christophe Lyon wrote: On 11/07/2017 10:33 AM, Aldy Hernandez wrote: While poking around in the backwards threader I noticed that we bail if we have already seen a starting BB. /* Do not jump-thread twice from the same block. */ if (bitmap_bit_p (thread

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-02 Thread Christophe Lyon
On Sun, 1 Jul 2018 at 12:56, Aldy Hernandez wrote: > > > > On 06/29/2018 02:50 PM, Jeff Law wrote: > > [ Returning to another old patch... ] > > > > On 11/07/2017 10:33 AM, Aldy Hernandez wrote: > >> [One more time, but without rejected HTML mail, because apparently this > >> is my first post to g

Re: [patch] jump threading multiple paths that start from the same BB

2018-07-01 Thread Aldy Hernandez
On 06/29/2018 02:50 PM, Jeff Law wrote: [ Returning to another old patch... ] On 11/07/2017 10:33 AM, Aldy Hernandez wrote: [One more time, but without rejected HTML mail, because apparently this is my first post to gcc-patches *ever* ;-)]. Howdy! While poking around in the backwards threa

Re: [patch] jump threading multiple paths that start from the same BB

2018-06-29 Thread Jeff Law
[ Returning to another old patch... ] On 11/07/2017 10:33 AM, Aldy Hernandez wrote: > [One more time, but without rejected HTML mail, because apparently this > is my first post to gcc-patches *ever* ;-)]. > > Howdy! > > While poking around in the backwards threader I noticed that we bail if > we

Re: [patch] jump threading multiple paths that start from the same BB

2017-11-30 Thread Jeff Law
On 11/29/2017 11:49 AM, Aldy Hernandez wrote: > On 11/27/2017 06:27 PM, Jeff Law wrote: >> On 11/07/2017 10:33 AM, Aldy Hernandez wrote: > >>> Without further ado, here are my monumental, earth shattering >>> improvements: >>> >>> Conditional branches >>>     Without patch: 411846839709 >>>     Wi

Re: [patch] jump threading multiple paths that start from the same BB

2017-11-29 Thread Aldy Hernandez
On 11/27/2017 06:27 PM, Jeff Law wrote: On 11/07/2017 10:33 AM, Aldy Hernandez wrote: Without further ado, here are my monumental, earth shattering improvements: Conditional branches    Without patch: 411846839709    With    patch: 411831330316     %changed: -0.0037660% Number of inst

Re: [patch] jump threading multiple paths that start from the same BB

2017-11-27 Thread Jeff Law
On 11/07/2017 10:33 AM, Aldy Hernandez wrote: > [One more time, but without rejected HTML mail, because apparently this > is my first post to gcc-patches *ever* ;-)]. > > Howdy! > > While poking around in the backwards threader I noticed that we bail if > we have already seen a starting BB. > >