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.

Sign up for GitHub

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

Jump to bottom

Can I compile pluto without -fopenmp? #73

Closed
dpotop opened this issue Feb 24, 2020 · 2 comments
Closed

Can I compile pluto without -fopenmp? #73

dpotop opened this issue Feb 24, 2020 · 2 comments

Comments

@dpotop
Copy link

dpotop commented Feb 24, 2020

On MacOSX it seems that -fopenmp is not accepted as an option by clang-9.0. On the forums I found that openmp support is not good, this may be an explanation.

However, I can't see why the compilation of pluto uses -fopemp, and how to disable this feature.

BTW, if someone is interested, I have compilation instructions for fedora, including some hacks needed to fully compile.
@bondhugula
Copy link
Owner

Thanks for reporting this. There used to be a place (in framework.cpp) where we used OpenMP to parallelize/speed up some computation. That's now obsolete and this flag can be removed.

bondhugula added a commit that referenced this issue Feb 24, 2020
@bondhugula
Drop -fopenmp from pluto compile flags
2576740 
- fix issue #73

Reported by: dpotop

Signed-off-by: Uday Bondhugula <udayb@iisc.ac.in>
@bondhugula
Copy link
Owner

Fixed by 2576740

@dpotop dpotop closed this as completed Mar 8, 2020
@thisiscam thisiscam mentioned this issue Oct 30, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bondhugula @dpotop