Re: [Rd] [PATCH] Add fpicflags for Intel(R) Fortran Compiler (PR#8344)

From: Prof Brian Ripley <ripley_at_stats.ox.ac.uk>
Date: Wed 23 Nov 2005 - 14:30:50 GMT

I've some playing with the Intel compilers, currently only on ia32. As far as I can see Intel provides compilers for only two OSes and on 50% of those -fPIC is wrong so I do think this is really Linux-specific. I have put in a Linux-specific change to set FPICFLAGS, but that is the least of the problems I have found.

The reason there is not a problem with the C compiler is that configure reports

         checking whether we are using the GNU C compiler... yes

and so configure takes the builtin defaults for gcc. This seems to come from the test of

int
main ()
{
#ifndef __GNUC__

        choke me
#endif

   ;
   return 0;
}

and so it seems that the masquerading by icc is intentional. This has some consequences: for example package foreign assumes that GCC accepts -Wno-long-long, but icc does not act on it. More seriously, it means that the default CFLAGS get set to "-g -O2", which is not what is documented. And with those flags, the build fails, incorrectly reporting that a regexp is invalid.

There are also problems with optimizing src/modules/dlamc.f under ifort.

On Tue, 22 Nov 2005, Danny van Dyk wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Dear Prof. Ripley,
>
> Prof Brian Ripley schrieb:
> | 1) The documented way to specify this is to set FPICFLAGS in
> | config.site: see the example in R-admin for the PG compilers. When you
> | tried that, what went wrong?
> Nothing went wrong. However, I was of the opinion that support of
> ifc/ifort 'out-of-the-box' is worthwhile. This is why I asked to include
> it into the source-tree. As a matter of fact, I stumbled upon this when
> testing Gentoo's R-2.2.0 buildscripts and added a similar patch to our
> repository.
>
> | 2) It seems strange to specify this for the Fortran compiler and not the
> | C or C++ compiler.
> I used the combination of 'gcc/g++/ifort' to build and test R. Testing
> the combination of 'icc/ifort' on Gentoo is scheduled already.
>
> | 3) You have suggested the change to a section for all OSes. Do you know
> | for sure that all compilers called 'ifort' on all OSes need the -fPIC
> | flag? Or is it really just for Linux (and AFAICS, just x86_64 Linux, as
> | i386 Linux seems to work with icc/ifort without it).
> a) For building shared ELF-libraries on x86_64 and ia64, you need to
> instruct the compiler to build PIC. As a matter of fact, this isn't
> necessary for x86, but it is strongly recommended as it should improve
> performance due to the lack of text relocations the dynamic linker would
> have to perform otherwise. This affects all OSes that can handle ELF
> binaries (Linux, FreeBSD, Solaris, tbc.).
>
> b) I surely can't guarantee that there never will be a different fortran
> compiler with same name on any OS. However, I think that it is a safe
> assumption to say that there currently is no such compiler on OSes
> supported by autotools.
>
>
> The number of target OSes and and supported ISAs makes it - in my eyes -
> worthwhile to add the check to the 'general' section of the configure.ac
> script. If - in a rare case - a conflict with other compiler emerged, it
> would still be possible to work around a problem in the following
> section of the script: '## Step 3. Individual platform overrides.'.
> (Emphasis here on 'individual' and 'override')
>
> Danny
> - --
> Danny van Dyk <kugelfang@gentoo.org>
> Gentoo/AMD64 Project, Gentoo Scientific Project
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.1 (GNU/Linux)
>
> iD8DBQFDg120aVNL8NrtU6IRAqVFAJ9Nv3FnfUjzGU2q4FEf+3TRek2HTQCfQG4R
> DD5dVT076/1HBJw5B4N+mck=
> =eBBz
> -----END PGP SIGNATURE-----
>
>

-- 
Brian D. Ripley,                  ripley@stats.ox.ac.uk
Professor of Applied Statistics,  http://www.stats.ox.ac.uk/~ripley/
University of Oxford,             Tel:  +44 1865 272861 (self)
1 South Parks Road,                     +44 1865 272866 (PA)
Oxford OX1 3TG, UK                Fax:  +44 1865 272595

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel
Received on Thu Nov 24 01:49:38 2005

This archive was generated by hypermail 2.1.8 : Mon 20 Feb 2006 - 03:21:34 GMT