Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 7:17 AM, H.J. Lu wrote: > On Fri, May 13, 2016 at 5:51 AM, Martin Liška wrote: >> On 05/13/2016 02:46 PM, Richard Biener wrote: >>> Use them for HOST_WIDE_INT printing, for [u]int64_t use the PRI stuff. >>> >>> Richard. >> >> Thanks you both, installed as r236208. >> > > I

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 5:51 AM, Martin Liška wrote: > On 05/13/2016 02:46 PM, Richard Biener wrote: >> Use them for HOST_WIDE_INT printing, for [u]int64_t use the PRI stuff. >> >> Richard. > > Thanks you both, installed as r236208. > It isn't fixed: /export/gnu/import/git/sources/gcc/gcc/tree-s

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Martin Liška
On 05/13/2016 02:46 PM, Richard Biener wrote: > Use them for HOST_WIDE_INT printing, for [u]int64_t use the PRI stuff. > > Richard. Thanks you both, installed as r236208. Martin

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Richard Biener
On Fri, May 13, 2016 at 2:43 PM, Kyrill Tkachov wrote: > Hi Martin, > > > On 13/05/16 13:39, Martin Liška wrote: >> >> On 05/13/2016 02:11 PM, H.J. Lu wrote: >>> >>> On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: On 05/13/2016 11:43 AM, Bin.Cheng wrote: > > On Thu, May 12,

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Kyrill Tkachov
Hi Martin, On 13/05/16 13:39, Martin Liška wrote: On 05/13/2016 02:11 PM, H.J. Lu wrote: On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: On 05/13/2016 11:43 AM, Bin.Cheng wrote: On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: On 05/12/2016 03:51 PM, Bin.Cheng wrote: On Thu, May

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Martin Liška
On 05/13/2016 02:11 PM, H.J. Lu wrote: > On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: >> On 05/13/2016 11:43 AM, Bin.Cheng wrote: >>> On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: On 05/12/2016 03:51 PM, Bin.Cheng wrote: > On Thu, May 12, 2016 at 1:13 PM, Martin Liška wro

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread H.J. Lu
On Fri, May 13, 2016 at 3:44 AM, Martin Liška wrote: > On 05/13/2016 11:43 AM, Bin.Cheng wrote: >> On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: >>> On 05/12/2016 03:51 PM, Bin.Cheng wrote: On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: > On 05/10/2016 03:16 PM, Bin.Cheng w

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Martin Liška
On 05/13/2016 11:43 AM, Bin.Cheng wrote: > On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: >> On 05/12/2016 03:51 PM, Bin.Cheng wrote: >>> On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: On 05/10/2016 03:16 PM, Bin.Cheng wrote: > Another way is to remove the use of id for struc

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-13 Thread Bin.Cheng
On Thu, May 12, 2016 at 5:41 PM, Martin Liška wrote: > On 05/12/2016 03:51 PM, Bin.Cheng wrote: >> On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: >>> On 05/10/2016 03:16 PM, Bin.Cheng wrote: Another way is to remove the use of id for struct iv_inv_expr_ent once for all. We can ch

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-12 Thread Martin Liška
On 05/12/2016 03:51 PM, Bin.Cheng wrote: > On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: >> On 05/10/2016 03:16 PM, Bin.Cheng wrote: >>> Another way is to remove the use of id for struct iv_inv_expr_ent once >>> for all. We can change iv_ca.used_inv_expr and cost_pair.inv_expr_id >>> to po

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-12 Thread Bin.Cheng
On Thu, May 12, 2016 at 1:13 PM, Martin Liška wrote: > On 05/10/2016 03:16 PM, Bin.Cheng wrote: >> Another way is to remove the use of id for struct iv_inv_expr_ent once >> for all. We can change iv_ca.used_inv_expr and cost_pair.inv_expr_id >> to pointers, and rename iv_inv_expr_ent.id to count

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-12 Thread Martin Liška
I'm going to test the patch. Thoughts? Martin >From ce02c80c053c2a8a63ce6e87f5779a8dc5f470ee Mon Sep 17 00:00:00 2001 From: marxin Date: Mon, 25 Apr 2016 14:29:01 +0200 Subject: [PATCH 3/3] Enhance dumps of IVOPTS gcc/ChangeLog: 2016-05-12 Martin Liska * tree-ssa-loop-ivopts.c (avg_loop_niter): Fix coding

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-11 Thread Martin Liška
On 05/10/2016 03:16 PM, Bin.Cheng wrote: > Another way is to remove the use of id for struct iv_inv_expr_ent once > for all. We can change iv_ca.used_inv_expr and cost_pair.inv_expr_id > to pointers, and rename iv_inv_expr_ent.id to count and use this to > record reference number in iv_ca. This i

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-10 Thread Bin.Cheng
On Mon, May 9, 2016 at 10:46 AM, Richard Biener wrote: > On Fri, May 6, 2016 at 11:19 AM, Martin Liška wrote: >> Hi. >> >> Honza asked me to explain the change more verbosely. >> The patch simplify enhances verbose dump of IVOPTS so that >> # of iterations is printed. Apart from that it also prin

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-09 Thread Richard Biener
On Fri, May 6, 2016 at 11:19 AM, Martin Liška wrote: > Hi. > > Honza asked me to explain the change more verbosely. > The patch simplify enhances verbose dump of IVOPTS so that > # of iterations is printed. Apart from that it also prints > invariant expression that are used during the algorithm wh

Re: [PATCH 3/3] Enhance dumps of IVOPTS

2016-05-06 Thread Martin Liška
Hi. Honza asked me to explain the change more verbosely. The patch simplify enhances verbose dump of IVOPTS so that # of iterations is printed. Apart from that it also prints invariant expression that are used during the algorithm which considers a set of candidates which is improved. Main motiv

[PATCH 3/3] Enhance dumps of IVOPTS

2016-04-29 Thread marxin
gcc/ChangeLog: 2016-04-25 Martin Liska * tree-ssa-loop-ivopts.c (struct ivopts_data): Add inv_expr_map. (tree_ssa_iv_optimize_init): Initialize it. (get_expr_id): Assign expressions to the map. (iv_ca_dump): Dump invariant expressions. (create_new_ivs):