Re: [PR 80293] Don't totally-sRA char arrays

2017-04-20 Thread Martin Jambor
Hi, On Mon, Apr 17, 2017 at 09:35:18PM +0200, Martin Jambor wrote: > On Thu, Apr 13, 2017 at 08:48:38PM +0200, Richard Biener wrote: > > On April 13, 2017 7:41:29 PM GMT+02:00, Martin Jambor > > wrote: ... > > >Therefore, I have adapted the patch to allow char arrays for const > > >decls only

Re: [PR 80293] Don't totally-sRA char arrays

2017-04-20 Thread Richard Biener
On Mon, 17 Apr 2017, Martin Jambor wrote: > Hi, > > On Thu, Apr 13, 2017 at 08:48:38PM +0200, Richard Biener wrote: > > On April 13, 2017 7:41:29 PM GMT+02:00, Martin Jambor > > wrote: > > >Hi, > > > > > >On Wed, Apr 12, 2017 at 01:55:01PM +0200, Richard Biener wrote: > > >> On Wed, 12 Apr 2017

Re: [PR 80293] Don't totally-sRA char arrays

2017-04-17 Thread Martin Jambor
Hi, On Thu, Apr 13, 2017 at 08:48:38PM +0200, Richard Biener wrote: > On April 13, 2017 7:41:29 PM GMT+02:00, Martin Jambor wrote: > >Hi, > > > >On Wed, Apr 12, 2017 at 01:55:01PM +0200, Richard Biener wrote: > >> On Wed, 12 Apr 2017, Martin Jambor wrote: > >> > >> > Hi, > >> > > >> > the patch

Re: [PR 80293] Don't totally-sRA char arrays

2017-04-13 Thread Richard Biener
On April 13, 2017 7:41:29 PM GMT+02:00, Martin Jambor wrote: >Hi, > >On Wed, Apr 12, 2017 at 01:55:01PM +0200, Richard Biener wrote: >> On Wed, 12 Apr 2017, Martin Jambor wrote: >> >> > Hi, >> > >> > the patch below is an attempt to deal with PR 80293 as >non-invasively >> > as possible. Basica

Re: [PR 80293] Don't totally-sRA char arrays

2017-04-13 Thread Martin Jambor
Hi, On Wed, Apr 12, 2017 at 01:55:01PM +0200, Richard Biener wrote: > On Wed, 12 Apr 2017, Martin Jambor wrote: > > > Hi, > > > > the patch below is an attempt to deal with PR 80293 as non-invasively > > as possible. Basically, it switches off total SRA scalarization of > > any local aggregates

Re: [PR 80293] Don't totally-sRA char arrays

2017-04-12 Thread Richard Biener
On Wed, 12 Apr 2017, Martin Jambor wrote: > Hi, > > the patch below is an attempt to deal with PR 80293 as non-invasively > as possible. Basically, it switches off total SRA scalarization of > any local aggregates which contains an array of elements that have one > byte (or less). > > The logic

[PR 80293] Don't totally-sRA char arrays

2017-04-12 Thread Martin Jambor
Hi, the patch below is an attempt to deal with PR 80293 as non-invasively as possible. Basically, it switches off total SRA scalarization of any local aggregates which contains an array of elements that have one byte (or less). The logic behind this is that accessing such arrays element-wise usu