[PATCH,fortran]: fix bind(c) common block alignment
Steve Kargl
sgk@troutmask.apl.washington.edu
Mon Aug 6 17:41:00 GMT 2007
On Mon, Aug 06, 2007 at 11:17:01AM -0600, Christopher D. Rickett wrote:
>
> currently, the patch only changes the alignment if the common block is
> bind(c), which means the compiler should have already verified that the
> common block fields interop with C.
>
> as to why we weren't seeing it before, i do not know. the
> bind_c_coms.f90/bind_c_coms_driver.c illustrates on x86 linux that there
> is a difference in the alignment. the generated assembly shows it, but
> the linker didn't complain.
>
> if it is acceptable for all common blocks to choose a C compatible
> alignment, then the patch is trivial to change to do this. is that what
> people prefer?
>
One thing to keep in mind is derived types that appear in
common blocks. These have SEQUENCE, and I suspect the
current alignment guarantees that SEQUENCE gives expected
results.
--
Steve
More information about the Fortran
mailing list