Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support icc/icpc #35

Closed
wjmelements opened this issue May 10, 2015 · 1 comment
Closed

Support icc/icpc #35

wjmelements opened this issue May 10, 2015 · 1 comment

Comments

@wjmelements
Copy link

icc and icpc are the required compilers on several clusters including TACC Stampede, and I've had to make some adjustments every time I've tried to get it to compile.

Basic changes:
cds/cxx11_atomic.h needs to detect that icpc has the atomic header
icpc is missing certain features of the atomic header. This is a shortlist:

  • std::atomic_ typedefs
  • atomic_memory_fence and similar functions are not in std::

If you think this is a worthy feature I may have time to do it properly the week after next.

@wjmelements
Copy link
Author

The reason to prefer the Intel headers to the cds version is that the compiler might have optimizations in mind that it only applies to those definitions.
I'm sufficiently satisfied by the performance of the default that I have closed this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant