add an notice about minimum Intel / GCC version required to configure output
Cactus requires full C++ 11 support, which for the Intel compiler suite requires a new enough Intel C++ compiler as well as a new enough STL (from g++) to work. While configure tests for features it does not give a hint as to what compiler version is required. In particular for the Intel compiler whose default setup often uses an old system installed g++ STL this can be annoying.
It would thus be good if configure, at least when detecting an Intel compiler, could output a minimum known good combination of Intel and g++ compiler to work. This may be Intel 2017 + gcc 6. Gcc 6 is a known requirement and stated in one of the release notes. Intel 2017 may be the first intel compiler to list compatibility with gcc 6 include files. Known to not work are versions Intel 2015 (https://stackoverflow.com/questions/29371301/intel-c-compiler-what-is-highest-gcc-version-compatibility) and earlier.
The current minimum icpc version in simfactory option lists is intel 2017 or intel 2016 (on Caltech’s wheeler cluster which has not been tested for a while).
Comments (12)
-
reporter -
reporter Pull request to test for C++11 math for isnan is here: https://bitbucket.org/cactuscode/cactus/pull-requests/152
This is probably better than trying to rely on feature test macros since those may not really tell the truth due to bugs in c++ libraries.
-
reporter Please review.
-
reporter Issue
#2660was marked as a duplicate of this issue. -
reporter @Zach Etienne would you be interested to review?
-
Sure, but where is the PR?
-
reporter Please see comment https://bitbucket.org/einsteintoolkit/tickets/issues/2663/add-an-notice-about-minimum-intel-gcc#comment-65334887 further up in this ticket for the URL of the pull request.
-
reporter There is no strict version check one could do. Eg gcc-4.8 on my debian box compiles the ET fine, but gcc-4.8 on an old RedHat based cluster fails. Either Debian includes some fixes in gcc or it is due to more modern system include files. In any case, sometimes even gcc-4.8 is fine, but not always.
-
reporter @Zach Etienne would you be interested to review? Pull request is earlier in this ticket and also, for convenience, here:
-
reporter @Zach Etienne says (in pull request): please apply.
-
reporter -
reporter - changed status to resolved
Thank you.
- Log in to comment
2017 claims gcc 6 support here: https://www.intel.com/content/www/us/en/developer/articles/release-notes/intel-c-compiler-170-for-linux-release-notes-for-intel-parallel-studio-xe-2017.html#sysreq