On Dec 10, 2013, at 10:42 AM, Dave Love <d.love_at_[hidden]> wrote:
> This doesn't seem to have been fixed, and I think it's going to bite
> here. Is this the right change?
Thanks for reminding us.
> --- openmpi-1.6.5/ompi/config/ompi_setup_mpi_fortran.m4~ 2012-04-03 15:30:24.000000000 +0100
> +++ openmpi-1.6.5/ompi/config/ompi_setup_mpi_fortran.m4 2013-12-10 12:23:54.232854527 +0000
> @@ -127,8 +127,8 @@
> AC_MSG_RESULT([skipped (no Fortran bindings)])
> bytes=`expr 4 \* $ac_cv_sizeof_int + $ac_cv_sizeof_size_t`
> - num_integers=`expr $bytes / $OMPI_SIZEOF_FORTRAN_INTEGER`
> - sanity=`expr $num_integers \* $OMPI_SIZEOF_FORTRAN_INTEGER`
> + num_integers=`expr $bytes / $ac_cv_sizeof_int`
> + sanity=`expr $num_integers \* $ac_cv_sizeof_int`
I think this is right, but it is has different implications for different series:
1. No more releases are planned for the v1.6 series. We can commit this fix over there, and it will be available via nightly tarballs. There are also ABI implications -- see #2, below.
2. This fix changes the ABI for the 1.5/1.6 and 1.7/1.8 series (separately, of course). As such, we will need to make this a non-default configure option. E.g., only do this new behavior if --enable-abi-breaking-fortran-status-i8-fix is specified (or some name like that). By default, we have to keep the ABI for the entire 1.5/1.6 and 1.7/1.8 series -- so if you specify this switch, you acknowledge that you're breaking ABI for the -i8 case.
3. For the v1.9 series (i.e., currently the SVN trunk), we can make this be the default, and the --enable-abi-breaking... switch will not exist.
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/