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

#pragma omp compile error #2

Closed
lambdaji opened this issue Jun 27, 2019 · 7 comments
Closed

#pragma omp compile error #2

lambdaji opened this issue Jun 27, 2019 · 7 comments
Labels
good first issue Good for newcomers

Comments

@lambdaji
Copy link

byteps/common/cpu_reducer.cc:64:26: error: expected '#pragma omp' clause before 'simd'

@bobzhuyb
Copy link
Member

Would you provide us more information about your environment? Are you building byteps using our dockerfile? If not, what is your gcc version?

This is related to the OpenMP version. We have tested gcc 4.9 and above, which includes OpenMP 4.0. I guess we'll need a flag to detect gcc version and choose omp clause accordingly.

@byronyi
Copy link
Member

byronyi commented Jun 27, 2019

byteps/common/cpu_reducer.cc:64:26: error: expected '#pragma omp' clause before 'simd'

Mind to share your environment, e.g. OS, compiler version, CUDA version etc.?

@lambdaji
Copy link
Author

lambdaji commented Jun 27, 2019

gcc version 4.8.5 20150623 (Red Hat 4.8.5-36) (GCC)
CUDA: NVIDIA_CUDA-9.0

@byronyi
Copy link
Member

byronyi commented Jun 27, 2019

Could you yum install devtoolset-7?

@lambdaji
Copy link
Author

Could you yum install devtoolset-7?

It works, thank you

@bobzhuyb bobzhuyb added the good first issue Good for newcomers label Jun 27, 2019
@bobzhuyb
Copy link
Member

Okay. Good to know. In general, we recommend starting with our dockerfiles, so that you will have less version issues.

If not, you must have gcc>=4.9.

We may add a compilation flag to disable OpenMP in the future, since it is not always needed by BytePS.

@byronyi
Copy link
Member

byronyi commented Jun 27, 2019

Could you yum install devtoolset-7?

It works, thank you

Good to know. I will close for now. Let us know if you have any other issues!

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

No branches or pull requests

3 participants