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'