MoL fails to build with Intel compiler

Comments (11)

1. 

   Ian Hinder reporter

   • changed status to open
   • removed comment

   • 2017-02-13T10:24:23+00:00
2. 

   Erik Schnetter

   • removed comment

   I commented out the parallel for constructs because this improves performance. If the above doesn't work, the commenting out the whole line (or outright deleting it) will work.

   I have just finished experimenting more with better parallelization in MoL, and I now know what to do. I plan to suggest a patch that applies this to all of MoL soon. If I make next Monday's call, we should discuss then.

   • 2017-02-13T12:41:09+00:00
3. 

   Frank Löffler

   • removed comment

   A question to the patch that introduces this: lot's of the "omp parallel for" lines seem to have been changed to "omp parallel simd", except that one line where the "simd" is missing. Was that on purpose?

   As a remark: "omp parallel simd" is an addition made in openmp 4. Which version of openmp should the Einstein Toolkit require? And connected to that: does anybody know how well this version is supported across machines?

   • 2017-02-13T14:31:44+00:00
4. 

   Erik Schnetter

   • removed comment

   Here is what I found to be most efficient: - all the loops that touch only a single grid function should have a simd - the OpenMP parallelization should be over the grid functions instead. I might have made a mistake in my earlier push. (In my defence, the performance improvement I saw was real.) A new patch should probably be properly reviewed, to avoid the problems I caused last time.

   I don't know which OpenMP version is supported in general. These days I tend to use recent versions of GCC instead of relying on older versions of Intel. We might want to autoconf simd.

   • 2017-02-13T15:39:48+00:00
5. 

   Ian Hinder reporter

   • removed comment

   In the mean time, would it be possible to make the minimal necessary change to make it build on intel? e.g. commenting out the entire line, or adding back in the "parallel" (though I haven't tested that on GCC, and don't know the openmp details).

   • 2017-02-14T00:57:29+00:00
6. 

   Ian Hinder reporter

   • removed comment

   I have removed the offending line completely in fc346d. Erik, please can you check that the effect is what you intended? If so, please close this ticket.

   The ET builds again with the Intel compiler. I have not looked at performance.

   • 2017-02-22T08:26:12+00:00
7. 

   Ian Hinder reporter

   • marked as
   • removed comment

   • 2017-02-22T08:26:52+00:00
8. 

   Ian Hinder reporter

   • changed status to open
   • removed comment

   • 2017-02-22T08:27:11+00:00
9. 

   Ian Hinder reporter

   • changed milestone to ET_2018_02
   • removed comment

   • 2018-01-22T08:56:01+00:00
10. 

    Ian Hinder reporter

    • changed status to resolved
    • removed comment

    Erik approves of the change.

    • 2018-01-22T09:43:40+00:00
11. 

    Roland Haas

    • edited description
    • changed status to closed

    • 2019-02-21T19:59:26+00:00
12. Log in to comment