Re: increase alignment of global structs in increase_alignment pass

2016-05-20 Thread Prathamesh Kulkarni
On 19 May 2016 at 13:19, Richard Biener wrote: > On Thu, 19 May 2016, Prathamesh Kulkarni wrote: > >> On 18 May 2016 at 19:38, Richard Biener wrote: >> > On Wed, 18 May 2016, Prathamesh Kulkarni wrote: >> > >> >> On 17 May 2016 at 18:36, Richard Biener wrote: >> >> > On Wed, 11 May 2016, Pratham

Re: increase alignment of global structs in increase_alignment pass

2016-05-19 Thread Richard Biener
On Thu, 19 May 2016, Prathamesh Kulkarni wrote: > On 18 May 2016 at 19:38, Richard Biener wrote: > > On Wed, 18 May 2016, Prathamesh Kulkarni wrote: > > > >> On 17 May 2016 at 18:36, Richard Biener wrote: > >> > On Wed, 11 May 2016, Prathamesh Kulkarni wrote: > >> > > >> >> On 6 May 2016 at 17:2

Re: increase alignment of global structs in increase_alignment pass

2016-05-18 Thread Prathamesh Kulkarni
On 18 May 2016 at 19:38, Richard Biener wrote: > On Wed, 18 May 2016, Prathamesh Kulkarni wrote: > >> On 17 May 2016 at 18:36, Richard Biener wrote: >> > On Wed, 11 May 2016, Prathamesh Kulkarni wrote: >> > >> >> On 6 May 2016 at 17:20, Richard Biener wrote: >> >> > >> >> > You can't simply use

Re: increase alignment of global structs in increase_alignment pass

2016-05-18 Thread Richard Biener
On Wed, 18 May 2016, Prathamesh Kulkarni wrote: > On 17 May 2016 at 18:36, Richard Biener wrote: > > On Wed, 11 May 2016, Prathamesh Kulkarni wrote: > > > >> On 6 May 2016 at 17:20, Richard Biener wrote: > >> > > >> > You can't simply use > >> > > >> > + offset = int_byte_position (field);

Re: increase alignment of global structs in increase_alignment pass

2016-05-18 Thread Prathamesh Kulkarni
On 17 May 2016 at 18:36, Richard Biener wrote: > On Wed, 11 May 2016, Prathamesh Kulkarni wrote: > >> On 6 May 2016 at 17:20, Richard Biener wrote: >> > >> > You can't simply use >> > >> > + offset = int_byte_position (field); >> > >> > as it can be placed at variable offset which will make

Re: increase alignment of global structs in increase_alignment pass

2016-05-17 Thread Richard Biener
On Wed, 11 May 2016, Prathamesh Kulkarni wrote: > On 6 May 2016 at 17:20, Richard Biener wrote: > > > > You can't simply use > > > > + offset = int_byte_position (field); > > > > as it can be placed at variable offset which will make int_byte_position > > ICE. Note it also returns a truncat

Re: increase alignment of global structs in increase_alignment pass

2016-05-17 Thread Prathamesh Kulkarni
ping https://gcc.gnu.org/ml/gcc/2016-05/msg00120.html Thanks, Prathamesh On 11 May 2016 at 15:39, Prathamesh Kulkarni wrote: > On 6 May 2016 at 17:20, Richard Biener wrote: >> On Wed, 4 May 2016, Prathamesh Kulkarni wrote: >> >>> On 23 February 2016 at 21:49, Prathamesh Kulkarni >>> wrote: >>>

Re: increase alignment of global structs in increase_alignment pass

2016-05-11 Thread Prathamesh Kulkarni
On 6 May 2016 at 17:20, Richard Biener wrote: > On Wed, 4 May 2016, Prathamesh Kulkarni wrote: > >> On 23 February 2016 at 21:49, Prathamesh Kulkarni >> wrote: >> > On 23 February 2016 at 17:31, Richard Biener wrote: >> >> On Tue, 23 Feb 2016, Prathamesh Kulkarni wrote: >> >> >> >>> On 22 Februa

Re: increase alignment of global structs in increase_alignment pass

2016-05-06 Thread Richard Biener
On Wed, 4 May 2016, Prathamesh Kulkarni wrote: > On 23 February 2016 at 21:49, Prathamesh Kulkarni > wrote: > > On 23 February 2016 at 17:31, Richard Biener wrote: > >> On Tue, 23 Feb 2016, Prathamesh Kulkarni wrote: > >> > >>> On 22 February 2016 at 17:36, Richard Biener wrote: > >>> > On Mon,

Re: increase alignment of global structs in increase_alignment pass

2016-05-04 Thread Prathamesh Kulkarni
On 23 February 2016 at 21:49, Prathamesh Kulkarni wrote: > On 23 February 2016 at 17:31, Richard Biener wrote: >> On Tue, 23 Feb 2016, Prathamesh Kulkarni wrote: >> >>> On 22 February 2016 at 17:36, Richard Biener wrote: >>> > On Mon, 22 Feb 2016, Prathamesh Kulkarni wrote: >>> > >>> >> Hi Richa

Re: increase alignment of global structs in increase_alignment pass

2016-02-24 Thread Prathamesh Kulkarni
On 23 February 2016 at 22:11, Marek Polacek wrote: > On Tue, Feb 23, 2016 at 09:49:37PM +0530, Prathamesh Kulkarni wrote: > >> diff --git a/gcc/tree-vectorizer.c b/gcc/tree-vectorizer.c >> index 2b25b45..a6af535 100644 >> --- a/gcc/tree-vectorizer.c >> +++ b/gcc/tree-vectorizer.c >> @@ -794,6 +794

Re: increase alignment of global structs in increase_alignment pass

2016-02-23 Thread Marek Polacek
On Tue, Feb 23, 2016 at 09:49:37PM +0530, Prathamesh Kulkarni wrote: > diff --git a/gcc/tree-vectorizer.c b/gcc/tree-vectorizer.c > index 2b25b45..a6af535 100644 > --- a/gcc/tree-vectorizer.c > +++ b/gcc/tree-vectorizer.c > @@ -794,6 +794,75 @@ make_pass_slp_vectorize (gcc::context *ctxt) >

Re: increase alignment of global structs in increase_alignment pass

2016-02-23 Thread Prathamesh Kulkarni
On 23 February 2016 at 17:31, Richard Biener wrote: > On Tue, 23 Feb 2016, Prathamesh Kulkarni wrote: > >> On 22 February 2016 at 17:36, Richard Biener wrote: >> > On Mon, 22 Feb 2016, Prathamesh Kulkarni wrote: >> > >> >> Hi Richard, >> >> As discussed in private mail, this version of patch atte

Re: increase alignment of global structs in increase_alignment pass

2016-02-23 Thread Richard Biener
On Tue, 23 Feb 2016, Prathamesh Kulkarni wrote: > On 22 February 2016 at 17:36, Richard Biener wrote: > > On Mon, 22 Feb 2016, Prathamesh Kulkarni wrote: > > > >> Hi Richard, > >> As discussed in private mail, this version of patch attempts to > >> increase alignment > >> of global struct decl if

Re: increase alignment of global structs in increase_alignment pass

2016-02-23 Thread Prathamesh Kulkarni
On 22 February 2016 at 17:36, Richard Biener wrote: > On Mon, 22 Feb 2016, Prathamesh Kulkarni wrote: > >> Hi Richard, >> As discussed in private mail, this version of patch attempts to >> increase alignment >> of global struct decl if it contains an an array field(s) and array's >> offset is a mu

Re: increase alignment of global structs in increase_alignment pass

2016-02-22 Thread Richard Biener
On Mon, 22 Feb 2016, Prathamesh Kulkarni wrote: > Hi Richard, > As discussed in private mail, this version of patch attempts to > increase alignment > of global struct decl if it contains an an array field(s) and array's > offset is a multiple of the alignment of vector type corresponding to > it'

increase alignment of global structs in increase_alignment pass

2016-02-22 Thread Prathamesh Kulkarni
Hi Richard, As discussed in private mail, this version of patch attempts to increase alignment of global struct decl if it contains an an array field(s) and array's offset is a multiple of the alignment of vector type corresponding to it's scalar type and recursively checks for nested structs. eg: