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

Need to add OpenMP flag. #54

Closed
patrick-kidger opened this issue Sep 9, 2020 · 1 comment
Closed

Need to add OpenMP flag. #54

patrick-kidger opened this issue Sep 9, 2020 · 1 comment

Comments

@patrick-kidger
Copy link
Collaborator

setup.py doesn't currently enable OpenMP for the C++ extensions. (Which by-the-by we may wish to think about the future of as maintaining those is going to be rather a hassle.)

See e.g.
https://github.com/patrick-kidger/signatory/blob/master/setup.py#L33
for an example of how to enable the OpenMP flags.

(Whilst we're at it we should also add the fvisibility flag; see the same example.)

@patrick-kidger
Copy link
Collaborator Author

Closing as we don't currently use C++.

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