Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-04 Thread Martin Liška
;>> Martin >>>> >>>>> >>>>>> Martin >>>>>> >>>> >> >> Hi Richard. >> >> There's the patch we talked about yesterday. It contains a few modification >> that >>

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-04 Thread Richard Biener
On Wed, Nov 4, 2015 at 11:38 AM, Martin Liška wrote: > On 11/03/2015 03:44 PM, Richard Biener wrote: >> On Tue, Nov 3, 2015 at 3:13 PM, Martin Liška wrote: >>> On 11/03/2015 02:46 PM, Richard Biener wrote: On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: > On 10/30/2015 01:13 PM, Ri

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-04 Thread Martin Liška
On 11/03/2015 03:44 PM, Richard Biener wrote: > On Tue, Nov 3, 2015 at 3:13 PM, Martin Liška wrote: >> On 11/03/2015 02:46 PM, Richard Biener wrote: >>> On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: On 10/30/2015 01:13 PM, Richard Biener wrote: > So I suggest to do the push/pop of

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-03 Thread Richard Biener
On Tue, Nov 3, 2015 at 3:13 PM, Martin Liška wrote: > On 11/03/2015 02:46 PM, Richard Biener wrote: >> On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: >>> On 10/30/2015 01:13 PM, Richard Biener wrote: So I suggest to do the push/pop of cfun there. do_per_function_toporder can be ma

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-03 Thread Martin Liška
; /* When in_dummy_function, we do have a cfun but current_function_decl is NULL. We also allow pushing NULL cfun and subsequently changing If you are fine with that, looks we've fixed all issues related to the change, right? Updated version of the is attached. Martin >

Re: [PATCH] Pass manager: add support for termination of pass list

2015-11-03 Thread Richard Biener
On Fri, Oct 30, 2015 at 1:53 PM, Martin Liška wrote: > On 10/30/2015 01:13 PM, Richard Biener wrote: >> So I suggest to do the push/pop of cfun there. >> do_per_function_toporder can be made static btw. >> >> Richard. > > Right, I've done that and it works (bootstrap has been currently running), >

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-30 Thread Martin Liška
[PATCH] Pass manager: add support for termination of pass list gcc/ChangeLog: 2015-10-30 Martin Liska * passes.c (do_per_function_toporder): Push to cfun before calling the pass manager. (execute_one_pass): Handle TODO_discard_function. (execute_pass_list_1): Terminate if

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-30 Thread Richard Biener
On Fri, Oct 30, 2015 at 12:59 PM, Martin Liška wrote: > On 10/30/2015 09:54 AM, Richard Biener wrote: >> On Thu, Oct 29, 2015 at 3:50 PM, Martin Liška wrote: >>> On 10/29/2015 02:15 PM, Richard Biener wrote: On Thu, Oct 29, 2015 at 10:49 AM, Martin Liška wrote: > On 10/28/2015 04:23 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-30 Thread Martin Liška
On 10/30/2015 09:54 AM, Richard Biener wrote: > On Thu, Oct 29, 2015 at 3:50 PM, Martin Liška wrote: >> On 10/29/2015 02:15 PM, Richard Biener wrote: >>> On Thu, Oct 29, 2015 at 10:49 AM, Martin Liška wrote: On 10/28/2015 04:23 PM, Richard Biener wrote: > On Tue, Oct 27, 2015 at 4:30 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-30 Thread Richard Biener
On Thu, Oct 29, 2015 at 3:50 PM, Martin Liška wrote: > On 10/29/2015 02:15 PM, Richard Biener wrote: >> On Thu, Oct 29, 2015 at 10:49 AM, Martin Liška wrote: >>> On 10/28/2015 04:23 PM, Richard Biener wrote: On Tue, Oct 27, 2015 at 4:30 PM, Martin Liška wrote: > On 10/27/2015 03:49 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-29 Thread Martin Liška
On 10/29/2015 02:15 PM, Richard Biener wrote: > On Thu, Oct 29, 2015 at 10:49 AM, Martin Liška wrote: >> On 10/28/2015 04:23 PM, Richard Biener wrote: >>> On Tue, Oct 27, 2015 at 4:30 PM, Martin Liška wrote: On 10/27/2015 03:49 PM, Richard Biener wrote: > On Tue, Oct 27, 2015 at 1:36 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-29 Thread Richard Biener
On Thu, Oct 29, 2015 at 10:49 AM, Martin Liška wrote: > On 10/28/2015 04:23 PM, Richard Biener wrote: >> On Tue, Oct 27, 2015 at 4:30 PM, Martin Liška wrote: >>> On 10/27/2015 03:49 PM, Richard Biener wrote: On Tue, Oct 27, 2015 at 1:36 PM, Martin Liška wrote: > On 10/26/2015 02:48 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-29 Thread Martin Liška
On 10/28/2015 04:23 PM, Richard Biener wrote: > On Tue, Oct 27, 2015 at 4:30 PM, Martin Liška wrote: >> On 10/27/2015 03:49 PM, Richard Biener wrote: >>> On Tue, Oct 27, 2015 at 1:36 PM, Martin Liška wrote: On 10/26/2015 02:48 PM, Richard Biener wrote: > On Thu, Oct 22, 2015 at 1:02 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-28 Thread Richard Biener
On Tue, Oct 27, 2015 at 4:30 PM, Martin Liška wrote: > On 10/27/2015 03:49 PM, Richard Biener wrote: >> On Tue, Oct 27, 2015 at 1:36 PM, Martin Liška wrote: >>> On 10/26/2015 02:48 PM, Richard Biener wrote: On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote: > On 10/21/2015 04:06 PM, R

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-27 Thread Martin Liška
On 10/27/2015 03:49 PM, Richard Biener wrote: > On Tue, Oct 27, 2015 at 1:36 PM, Martin Liška wrote: >> On 10/26/2015 02:48 PM, Richard Biener wrote: >>> On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote: On 10/21/2015 04:06 PM, Richard Biener wrote: > On Wed, Oct 21, 2015 at 1:24 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-27 Thread Richard Biener
On Tue, Oct 27, 2015 at 1:36 PM, Martin Liška wrote: > On 10/26/2015 02:48 PM, Richard Biener wrote: >> On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote: >>> On 10/21/2015 04:06 PM, Richard Biener wrote: On Wed, Oct 21, 2015 at 1:24 PM, Martin Liška wrote: > On 10/21/2015 11:59 AM, R

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-27 Thread Martin Liška
On 10/26/2015 02:48 PM, Richard Biener wrote: > On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote: >> On 10/21/2015 04:06 PM, Richard Biener wrote: >>> On Wed, Oct 21, 2015 at 1:24 PM, Martin Liška wrote: On 10/21/2015 11:59 AM, Richard Biener wrote: > On Wed, Oct 21, 2015 at 11:19 AM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-26 Thread Richard Biener
On Mon, Oct 26, 2015 at 2:48 PM, Richard Biener wrote: > On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote: >> On 10/21/2015 04:06 PM, Richard Biener wrote: >>> On Wed, Oct 21, 2015 at 1:24 PM, Martin Liška wrote: On 10/21/2015 11:59 AM, Richard Biener wrote: > On Wed, Oct 21, 2015 at

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-26 Thread Richard Biener
On Thu, Oct 22, 2015 at 1:02 PM, Martin Liška wrote: > On 10/21/2015 04:06 PM, Richard Biener wrote: >> On Wed, Oct 21, 2015 at 1:24 PM, Martin Liška wrote: >>> On 10/21/2015 11:59 AM, Richard Biener wrote: On Wed, Oct 21, 2015 at 11:19 AM, Martin Liška wrote: > On 10/20/2015 03:39 PM,

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-22 Thread Martin Liška
;push_cfun (fn); >execute_pass_list_1 (pass); > - if (fn->cfg) > + if (gimple_has_body_p (current_function_decl) && fn->cfg) > { > free_dominance_info (CDI_DOMINATORS); >free_dominance_info (CDI_POST_DOMINATORS); > > here you'

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-21 Thread Richard Biener
On Wed, Oct 21, 2015 at 1:24 PM, Martin Liška wrote: > On 10/21/2015 11:59 AM, Richard Biener wrote: >> On Wed, Oct 21, 2015 at 11:19 AM, Martin Liška wrote: >>> On 10/20/2015 03:39 PM, Richard Biener wrote: On Tue, Oct 20, 2015 at 3:00 PM, Martin Liška wrote: > Hello. > > As pa

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-21 Thread Martin Liška
ng the pass manager" is necessary after TODO_discard_function, yes. > But that may be simply done via a has_body () check then? Thanks, there's second version of the patch. I'm going to start regression tests. Martin > >> Thanks, >> Martin >> &g

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-21 Thread Richard Biener
On Wed, Oct 21, 2015 at 11:19 AM, Martin Liška wrote: > On 10/20/2015 03:39 PM, Richard Biener wrote: >> On Tue, Oct 20, 2015 at 3:00 PM, Martin Liška wrote: >>> Hello. >>> >>> As part of upcoming merge of HSA branch, we would like to have possibility >>> to terminate >>> pass manager after exec

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-21 Thread Martin Liška
On 10/20/2015 03:39 PM, Richard Biener wrote: > On Tue, Oct 20, 2015 at 3:00 PM, Martin Liška wrote: >> Hello. >> >> As part of upcoming merge of HSA branch, we would like to have possibility >> to terminate >> pass manager after execution of the HSA generation pass. The HSA back-end is >> imple

Re: [PATCH] Pass manager: add support for termination of pass list

2015-10-20 Thread Richard Biener
On Tue, Oct 20, 2015 at 3:00 PM, Martin Liška wrote: > Hello. > > As part of upcoming merge of HSA branch, we would like to have possibility to > terminate > pass manager after execution of the HSA generation pass. The HSA back-end is > implemented > as a tree pass that directly emits HSAIL from

[PATCH] Pass manager: add support for termination of pass list

2015-10-20 Thread Martin Liška
Hello. As part of upcoming merge of HSA branch, we would like to have possibility to terminate pass manager after execution of the HSA generation pass. The HSA back-end is implemented as a tree pass that directly emits HSAIL from gimple tree representation. The pass operates on clones created b