Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-12-05 Thread Jay Foad
On 3 December 2014 at 14:36, Martin Jambor wrote: > On Wed, Dec 03, 2014 at 10:53:54AM +, Jay Foad wrote: >> > Index: src/gcc/ipa-prop.h >> > === >> > --- src.orig/gcc/ipa-prop.h >> > +++ src/gcc/ipa-prop.h >> > @@ -144,6 +144,17

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-12-03 Thread Martin Jambor
Hi, thank you very much for pointing out all the spelling mistakes. I suppose you could tell I was writing this in a hurry :-) Nevertheless I should probably make a habit of spell checking stuff before sending it out. As far as your question is concerned: On Wed, Dec 03, 2014 at 10:53:54AM +0

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-12-03 Thread Jay Foad
> Index: src/gcc/ipa-cp.c > === > --- src.orig/gcc/ipa-cp.c > +++ src/gcc/ipa-cp.c > @@ -262,6 +262,9 @@ public: >ipcp_lattice ctxlat; >/* Lattices describing aggregate parts. */ >ipcp_agg_lattice *aggs; > + /* Alignment

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-12-02 Thread Martin Jambor
Hi, thanks for the comments. On Mon, Dec 01, 2014 at 11:40:59PM +0100, Jan Hubicka wrote: > > > 2014-11-19 Martin Jambor > > > > > > * ipa-prop.h (ipa_alignment): New type. > > > (ipa_jump_func): New field alignment. > > > (ipcp_transformation_summary) New type. > > > (ipcp_grow_trans

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-12-01 Thread Jan Hubicka
> > 2014-11-19 Martin Jambor > > > > * ipa-prop.h (ipa_alignment): New type. > > (ipa_jump_func): New field alignment. > > (ipcp_transformation_summary) New type. > > (ipcp_grow_transformations_if_necessary): Declare. > > (ipa_node_agg_replacements): Removed. > > (ipcp_t

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-12-01 Thread Martin Jambor
Ping (this is a new feature submitted in stage1 so I'd really appreciate a review before it is going to be deemed "too late"). Thanks, Martin On Wed, Nov 19, 2014 at 12:32:53AM +0100, Martin Jambor wrote: > Hi, > ... > > 2014-11-19 Martin Jambor > > * ipa-prop.h (ipa_alignment): N

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-11-18 Thread Martin Jambor
Hi, On Mon, Nov 17, 2014 at 01:05:23PM +0100, Richard Biener wrote: > On Sat, Nov 15, 2014 at 2:04 AM, Martin Jambor wrote: > > Hi, > > > > this patch adds very simple propagation of alignment of pointers to > > IPA-CP. Because I have not attempted to estimate profitability of > > such propagati

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-11-17 Thread Richard Biener
On Sat, Nov 15, 2014 at 2:04 AM, Martin Jambor wrote: > Hi, > > this patch adds very simple propagation of alignment of pointers to > IPA-CP. Because I have not attempted to estimate profitability of > such propagation in any way, it does not do any cloning, just > propagation when the alignment

Re: [PATCH] Make IPA-CP propagate alignment information of pointers

2014-11-16 Thread Toon Moene
On 11/15/2014 02:04 AM, Martin Jambor wrote: Hi, this patch adds very simple propagation of alignment of pointers to IPA-CP. Because I have not attempted to estimate profitability of such propagation in any way, it does not do any cloning, just propagation when the alignment is known and the s