Re: [Fortran-dev][Patch, Fortran] C_F_Pointer cleanup

2012-07-20 Thread Richard Guenther
On Fri, Jul 20, 2012 at 12:17 PM, Tobias Burnus wrote: > On 07/20/2012 11:52 AM, Mikael Morin wrote >> >> OK. > > > Thanks for the review. > > >> I guess the division will be dropped too (the middle-end does the >> reverse multiplication implicitly in the pointer arithmetic). >> I wonder which one

Re: [Fortran-dev][Patch, Fortran] C_F_Pointer cleanup

2012-07-20 Thread Tobias Burnus
On 07/20/2012 11:52 AM, Mikael Morin wrote OK. Thanks for the review. I guess the division will be dropped too (the middle-end does the reverse multiplication implicitly in the pointer arithmetic). I wonder which one is faster (the casts in the second version are certainly not nice for the mi

Re: [Fortran-dev][Patch, Fortran] C_F_Pointer cleanup

2012-07-20 Thread Mikael Morin
On 15/07/2012 08:23, Tobias Burnus wrote: > Dear all, > > that's a follow up cleanup to the patch, which has just been merged. > Most parts should be really obvious and nice, however, the offset part > isn't. As the offset is not part of the descriptor defined at DTS > 29113:2012's "8.2 C descript

[Fortran-dev][Patch, Fortran] C_F_Pointer cleanup

2012-07-14 Thread Tobias Burnus
Dear all, that's a follow up cleanup to the patch, which has just been merged. Most parts should be really obvious and nice, however, the offset part isn't. As the offset is not part of the descriptor defined at DTS 29113:2012's "8.2 C descriptors" (p. 17), we will have to drop it at some poi