Re: [PATCH] Fix PR61950

2014-08-14 Thread Richard Biener
On Thu, 14 Aug 2014, Paul Richard Thomas wrote: > Dear Richie and Tobias, > > I OK'd this patch on 9th August but I now see that the posting bounced > because of mime content emanating from my phone mail reader :-( > > I also thought that the patch is obvious. I already applied it after your ap

Re: [PATCH] Fix PR61950

2014-08-14 Thread Paul Richard Thomas
Dear Richie and Tobias, I OK'd this patch on 9th August but I now see that the posting bounced because of mime content emanating from my phone mail reader :-( I also thought that the patch is obvious. Cheers Paul On 13 August 2014 23:22, Tobias Burnus wrote: > Hi Richard, > > sorry for the b

Re: [PATCH] Fix PR61950

2014-08-13 Thread Tobias Burnus
Hi Richard, sorry for the belated reply due to spending lazily my time at the sea ... The patch is okay - and I would even claim that it is obvious. Regarding the data type, I have no idea why it uses a hard-coded 32bit integer instead of a size-type one. I have added it as item to https://gc

Re: [PATCH] Fix PR61950

2014-08-07 Thread Richard Biener
On Thu, 31 Jul 2014, Richard Biener wrote: > > The following fixes PR61950 by properly initializing the _size field > of the static constructor for the vtable init member. Previously > gfortran would use a 64bit integer to initialize the 32bit size field > (not sure why larger aggregates are not

[PATCH] Fix PR61950

2014-07-31 Thread Richard Biener
The following fixes PR61950 by properly initializing the _size field of the static constructor for the vtable init member. Previously gfortran would use a 64bit integer to initialize the 32bit size field (not sure why larger aggregates are not considered). This breaks more sophisticated constant