Thanks Ralf!

If it matters, as I understand it, the name pgfortran is meant to symbolize that the compiler now supports things beyond Fortran 90 -- it supports Fortran 95 and some Fortran 03 (and beyond?) features. So I don't know if you want to update the comment that specifically states F77 and F90 or not...


On Nov 14, 2009, at 2:35 AM, Ralf Wildenhues wrote:

Hi Jeff,

* Jeff Squyres wrote on Wed, Nov 11, 2009 at 09:45:53PM CET:
> Could Libtool be adjusted now to look for the "pgfortran" name first
> for all Fortran variants, and if not found, fall back to the old
> names?  That would seem to give flexibility for the future.

That name doesn't seem to be problematic, can't see any other use of it
out there.  I'm pushing this patch.

Thanks,
Ralf

    Recognize `pgfortran' as Portland compiler.

    * libltdl/m4/libtool.m4 (_LT_COMPILER_PIC, _LT_LINKER_SHLIBS):
    Accept `pgfortran*' too.
    Report by Jeff Squyres.

diff --git a/libltdl/m4/libtool.m4 b/libltdl/m4/libtool.m4
index 206940e..1f58a38 100644
--- a/libltdl/m4/libtool.m4
+++ b/libltdl/m4/libtool.m4
@@ -4042,7 +4042,7 @@ m4_if([$1], [CXX], [
        _LT_TAGVAR(lt_prog_compiler_pic, $1)='--shared'
        _LT_TAGVAR(lt_prog_compiler_static, $1)='--static'
        ;;
-      pgcc* | pgf77* | pgf90* | pgf95*)
+      pgcc* | pgf77* | pgf90* | pgf95* | pgfortran*)
         # Portland Group compilers (*not* the Pentium gcc compiler,
        # which looks to be a dead project)
        _LT_TAGVAR(lt_prog_compiler_wl, $1)='-Wl,'
@@ -4454,7 +4454,8 @@ _LT_EOF
_LT_TAGVAR(whole_archive_flag_spec, $1)='${wl}--whole- archive`for conv in $convenience\"\"; do test -n \"$conv\" && new_convenience=\"$new_convenience,$conv\"; done; func_echo_all \"$new_convenience\"` ${wl}--no-whole-archive'
          tmp_addflag=' $pic_flag'
          ;;
- pgf77* | pgf90* | pgf95*) # Portland Group f77 and f90 compilers
+       pgf77* | pgf90* | pgf95* | pgfortran*)
+ # Portland Group f77 and f90 compilers _LT_TAGVAR(whole_archive_flag_spec, $1)='${wl}--whole- archive`for conv in $convenience\"\"; do test -n \"$conv\" && new_convenience=\"$new_convenience,$conv\"; done; func_echo_all \"$new_convenience\"` ${wl}--no-whole-archive'
          tmp_addflag=' $pic_flag -Mnomain' ;;
        ecc*,ia64* | icc*,ia64*)        # Intel C compiler on ia64



--
Jeff Squyres
jsquy...@cisco.com



Reply via email to