[gmx-developers] FYI: compilation with Intel 11.0, 64 bit.

[Michael Shirts]

Some bits from compiling with intel/Compiler/11.0/083/bin/intel64/icc,
some of which may need to be addressed, others of which may not:

1. There were a number of warnings of the form:
icc: command line remark #10010: option '-mp' is deprecated and will
be removed in a future release. See '-help deprecated'

2.  Loads of warnings of the form.
"argument of type "const char *" is incompatible with parameter of type "void *"
in code like: if (strcasecmp(ptype,xl[j].entry) == 0)

In almost all cases, it's strcasecmp or strdup giving the warning.

3. In src/gmxlib/gmx_cyclecounter.c, src/mdlib/sim_util.c,
src/tools/gmx_pme_tune.c the line: struct timezone tz = {0,0}; throws
an "incomplete struct" error.  Looking at the manual for gettimeofday,
I found that the timezone struct was deprecated, it is not required to
be supported by libc, and it's use should be considered a bug:
replacing with NULL in calls to gettimeofday is the standard usage
(see "man getttimeofday").  Replacing all instances of &tz with NULL
(which were only in calls to gettimeofday) appears to solve the
problem.  I will check this into the main branch unless there's
something I'm missing here.

4. Dies on some // comments in a number of places (some of them mine
(opps!) but from other contributors as well.

5. src/ngmx/xrama.c needs to include "maths.h" to be able to compile
when M_PI is not defined.