Hi Christian, Markus,

* Christian Rössel wrote on Fri, Feb 18, 2011 at 03:57:30PM CET:
> AC_OPENMP returns "none needed" for the IBM xl Fortran compilers (xlf*,
> bgxlf*, mpixlf*). This is wrong, you need to use -qsmp=omp.
> 
> configure tries to compile following program:
> 
> |       program main
> |       call omp_get_num_threads
> |       end
> 
> Compilation succeeds without specifying any OpenMP flag. If you modify
> the test program to
> 
>       program main
>       implicit none
> !$    integer tid
>       tid = 42
>       call omp_set_num_threads(2)
>       end
> 
> the correct flags are detected.

Thanks for the bug report and patch.  I have one question (and I'd still
like to do some testing before pushing):  Isn't the '!' line a Fortran
90 comment, and doesn't that have to be written differently for pure
Fortran 77?

> See the attached patch where I also
> added OpenMP flags for the Cray and NEC compilers. If you are happy with
> this patch, please add Markus Geimer to THANKS, too.

Yes, that part is quite obviously good (it has a near-zero chance of
regressing anything!).

Thanks,
Ralf

Reply via email to