Warnings when compiling with icpc

[PrometheusPi]

As @QJohn2017 mentioned in #89 there are several warnings when icpc is used instead of open mpi. This should be resolved for future use.

The warnings for

mpic++ -Wall -O3 -lfftw3 -lm -D__PARALLEL_SETTING__=1 -c -fopenmp -lz main.cpp

were:

• math libray

icpc: warning #10314: specifying -lm before object files may supercede the Intel(R) math library and affect performance

• openMP flag

icpc: command line warning #10006: ignoring unknown option '-fopenmp'

• #pragma once

parallel_jobs.h(22): remark #1782: #pragma once is obsolete. Use #ifndef guard instead.
  #pragma once
              ^

parallel_jobs.h(54): remark #1418: external function definition with no prior declaration
  int start_array(int* numtasks,
      ^

parallel_jobs.h(107): remark #1418: external function definition with no prior declaration
  int end_array(void)
      ^

parallel_jobs.h(130): remark #1418: external function definition with no prior declaration
  int check_break(void)
      ^

all_directions.hpp(21): remark #1782: #pragma once is obsolete. Use #ifndef guard instead.
  #pragma once
              ^

main.cpp(71): remark #181: argument is incompatible with corresponding format string conversion
      printf("this is job %5d of %5d jobs in the array (on %s = rank: %d)\n", i, N_max, pHost, rank);
                                                                              ^

main.cpp(71): remark #181: argument is incompatible with corresponding format string conversion
      printf("this is job %5d of %5d jobs in the array (on %s = rank: %d)\n", i, N_max, pHost, rank);
                                                                                                     ^

and (as in all versions) in during:

g++ -Wall -O3 -lfftw3 -lm -c  -fopenmp -I./include/ single_direction.cpp

the warning:

In file included from single_direction.cpp:35:
run_through_data.hpp: In function ‘void run_through_data(const one_line*, unsigned int, DET) [with DET = Detector_fft*]’:
run_through_data.hpp:56: warning: ‘time_fill.Discrete<double>::future’ may be used uninitialized in this function
run_through_data.hpp:56: warning: ‘time_fill.Discrete<double>::now’ may be used uninitialized in this function
run_through_data.hpp:56: warning: ‘time_fill.Discrete<double>::old’ may be used uninitialized in this function

[PrometheusPi]

@QJohn2017
What is the version of icp you are using and what version is behind mpic++?

[QJohn2017]

@PrometheusPi
My icpc version is icpc (ICC) 11.1 20091012

[PrometheusPi]

@belfhi Could you tell me why you needed to include the

-L${HOME}/.local/lib/

in the makefile when using the ICC?

[PrometheusPi]

I assume your fftw is located there, is this correct?

[belfhi]

Yes, that was for the fftw library, but I didn't realize it was already installed on the cluster, so it itsn't neccessary.

[belfhi]

ldconfig -p | grep libfftw
libfftw3l_threads.so.3 (libc6,x86-64) => /lib64/libfftw3l_threads.so.3
libfftw3l_threads.so (libc6,x86-64) => /lib64/libfftw3l_threads.so
libfftw3l_omp.so.3 (libc6,x86-64) => /lib64/libfftw3l_omp.so.3
libfftw3l_omp.so (libc6,x86-64) => /lib64/libfftw3l_omp.so
libfftw3l.so.3 (libc6,x86-64) => /lib64/libfftw3l.so.3
libfftw3l.so (libc6,x86-64) => /lib64/libfftw3l.so
libfftw3f_threads.so.3 (libc6,x86-64) => /lib64/libfftw3f_threads.so.3
libfftw3f_threads.so (libc6,x86-64) => /lib64/libfftw3f_threads.so
libfftw3f_omp.so.3 (libc6,x86-64) => /lib64/libfftw3f_omp.so.3
libfftw3f_omp.so (libc6,x86-64) => /lib64/libfftw3f_omp.so
libfftw3f.so.3 (libc6,x86-64) => /lib64/libfftw3f.so.3
libfftw3f.so (libc6,x86-64) => /lib64/libfftw3f.so
libfftw3_threads.so.3 (libc6,x86-64) => /lib64/libfftw3_threads.so.3
libfftw3_threads.so (libc6,x86-64) => /lib64/libfftw3_threads.so
libfftw3_omp.so.3 (libc6,x86-64) => /lib64/libfftw3_omp.so.3
libfftw3_omp.so (libc6,x86-64) => /lib64/libfftw3_omp.so
libfftw3.so.3 (libc6,x86-64) => /lib64/libfftw3.so.3
libfftw3.so (libc6,x86-64) => /lib64/libfftw3.so
libfftw_threads.so.2 (libc6,x86-64) => /lib64/libfftw_threads.so.2
libfftw.so.2 (libc6,x86-64) => /lib64/libfftw.so.2

[PrometheusPi]

I got clara2 to compile with the icc and openMPI. I will test this further to get ride of warnings in as many versions as possible.

[PrometheusPi]

@QJohn2017 we have an Intel 11.1 compiler available at our cluster, I will check whether I can reproduce your warnings.

[QJohn2017]

@PrometheusPi Okay, thank you very much! I would really like to eliminate the warnings when compiling the code and hope to hear your good news.