-
Notifications
You must be signed in to change notification settings - Fork 3
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
Highlight where interventions shall be put in place if OMP is not available #23
Comments
On windows you get OpenMP support with MSVC from the start - no problems there. |
I understand that: great. |
@dpiparo based on your patch, the openmp compatibility issue is almost resolved. I could try to apply it and resolve other issues that may appear as well. The problem is that I dont have access to OSx. I tried openstack but it has only linux/ windows images. The only solution that I am aware of is to set up travis to compile/ run under Mac OS and do all the debugging there (quite inefficient). |
Hi @kiliakis, |
Yes, I was thinking of something like that. The changes that need to be made are not so many. I am afraid that there will be other problems as well but we'll see. |
Hi,
if OMP is not available, many issues will arise. This happens on OSx, for example. Given that compatibility with Windows is planned too, on that platform issues can arise as well.
The attached patch highlights the places in the CMake configuration and in the code that need attention.
NOTA BENE: this patch is not made to be applied as is, but rather as a roadmap to implement compatibility. In other words, only 50% of the work was done :)
make_it_work_on_osx.txt
The text was updated successfully, but these errors were encountered: