Re: [PATCH][RFC] Add gimple-cfg.h.

On 08/27/2018 12:00 PM, Richard Biener wrote: > On Mon, Aug 27, 2018 at 10:43 AM Martin Liška wrote: >> >> On 08/27/2018 09:56 AM, Richard Biener wrote: >>> On Mon, Aug 27, 2018 at 9:13 AM Martin Liška wrote: Hi. There's updated version with notes that were discussed with Rich

Re: [PATCH][RFC] Add gimple-cfg.h.

On Mon, Aug 27, 2018 at 10:43 AM Martin Liška wrote: > > On 08/27/2018 09:56 AM, Richard Biener wrote: > > On Mon, Aug 27, 2018 at 9:13 AM Martin Liška wrote: > >> > >> Hi. > >> > >> There's updated version with notes that were discussed with Richi > >> on IRC. As pointed out label_to_block can p

Re: [PATCH][RFC] Add gimple-cfg.h.

On 08/27/2018 09:56 AM, Richard Biener wrote: > On Mon, Aug 27, 2018 at 9:13 AM Martin Liška wrote: >> >> Hi. >> >> There's updated version with notes that were discussed with Richi >> on IRC. As pointed out label_to_block can potentially lead to >> quadratic complexity, but it's not ambition of t

Re: [PATCH][RFC] Add gimple-cfg.h.

On Mon, Aug 27, 2018 at 9:13 AM Martin Liška wrote: > > Hi. > > There's updated version with notes that were discussed with Richi > on IRC. As pointed out label_to_block can potentially lead to > quadratic complexity, but it's not ambition of the patch to resolve > that. Please don't @@ -3239,7

Re: [PATCH][RFC] Add gimple-cfg.h.

Hi. There's updated version with notes that were discussed with Richi on IRC. As pointed out label_to_block can potentially lead to quadratic complexity, but it's not ambition of the patch to resolve that. Martin >From 393a0b87d7d667f0db6e512a840f2da3f51c8f7b Mon Sep 17 00:00:00 2001 From: marxin

Re: [PATCH][RFC] Add gimple-cfg.h.

On 08/21/2018 03:55 PM, Richard Biener wrote: > On Tue, Aug 21, 2018 at 2:58 PM Martin Liška wrote: >> >> Hi. >> >> I'm considering adding couple of new gimple-related functions that >> are related to gswitch statement. >> >> Is it a good idea? > > Just add them to tree-cfg.h? That's what should

Re: [PATCH][RFC] Add gimple-cfg.h.

On Tue, Aug 21, 2018 at 2:58 PM Martin Liška wrote: > > Hi. > > I'm considering adding couple of new gimple-related functions that > are related to gswitch statement. > > Is it a good idea? Just add them to tree-cfg.h? That's what should be really called gimple-cfg.h these days... Richard. > M

[PATCH][RFC] Add gimple-cfg.h.

Hi. I'm considering adding couple of new gimple-related functions that are related to gswitch statement. Is it a good idea? Martin gcc/ChangeLog: 2018-08-06 Martin Liska * gimple-cfg.h: New file. * ipa-fnsummary.c (set_switch_stmt_execution_predicate): Use gimple_swi