Fortran 90 bindings

Created by: williamfgc

Created by: williamfgc

@chuckatkins I need you help as the Fortran test and example fail on Mac when MPI is enabled, it also creates the hello_bpWriter_f.dSYM directory under bin. Everything works fine on my Ubuntu box. See below:

$ ./bin/hello_bpWriter_f dyld: Library not loaded: /usr/local/opt/gcc/lib/gcc/6/libgfortran.3.dylib Referenced from: /usr/local/opt/open-mpi/lib/libmpi_usempif08.20.dylib Reason: image not found Trace/BPT trap: 5

Thanks!

Created by: williamfgc

This is also ready to go. Needed to resolve conflicts after merging with master.

Created by: chuckatkins

Let's add another line here do show the fortran compiler information:

if(ADIOS2_HAVE_Fortran)
  message("  Fortran Compiler : ${CMAKE_Fortran_COMPILER_ID} ${CMAKE_Fortran_COMPILER_VERSION} ${CMAKE_Fortran_COMPILER_WRAPPER}")
  message("    ${CMAKE_Fortran_COMPILER}")
endif()

Created by: chuckatkins

Rather than build the list of source files, go ahead and explicitly add them:

add_library(adios2_f
  adios2_mod.f90
  ...
  adios2_f2c.cpp
)
if(ADIOS2_HAVE_MPI)
  target_sources(adios2_f PRIVATE
    mpi/adios2_adios_init_mod.f90 
    mpi/adios2_io_open_mod.f90
  )
else()
  target_sources(adios2_f PRIVATE
    nompi/adios2_adios_init_nompi_mod.f90 
    nompi/adios2_io_open_nompi_mod.f90
  )
endif()

Created by: chuckatkins

Is there a compelling reason to give the enums explicit values rather than just let it be automatic? If so, they should all get them, not just this half.

Created by: chuckatkins

W.R.T to naming conventions, rather than dp for double precision, would it be more appropriate to use the real8 fortran convention? i.e.

...
adios2_write_real
adios2_write_real8
...
adios2_write_complex
adios2_write_complex8
...

Created by: chuckatkins

Try consolidating a bit:

add_executable(TestBPWriteTypes_f SmallTestData_mod.f90)

if(ADIOS2_HAVE_MPI)
  target_sources(TestBPWriteTypes_f PRIVATE TestBPWriteTypes.f90)
else()
  target_sources(TestBPWriteTypes_f PRIVATE TestBPWriteTypes_nompi.f90)
  target_include_directories(TestBPWriteTypes_f PRIVATE ${MPI_Fortran_INCLUDE_PATH})
  target_link_libraries(TestBPWriteTypes_f PRIVATE ${MPI_Fortran_LIBRARIES})
endif()

target_link_libraries(TestBPWriteTypes_f PRIVATE adios2_f)

Created by: williamfgc

Sadly, real*8 is non-standard (even though it's everywhere), and kind=8 is not guaranteed to be 8 bytes (although most compilers do), the preferred way in Fortran 90 is using a dp parameter: https://web.stanford.edu/class/me200c/tutorial_90/04_basic.html http://fortranwiki.org/fortran/show/Real+precision . Also, compilers allow complex*16 as being equal to complex(kind=8) http://earth.uni-muenster.de/~joergs/doc/f90/unix-um/dfum_035.html confusing, isn't it?

Created by: williamfgc

Good catch. Fortran integer parameter types must match the C enum ones. I can refactor it on the Fortran-C glue code with a switch to leave the C side untouched, but it would add another if else-if.

Created by: williamfgc

Good addition.