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

openmp & OSX 13.5 #1609

Closed
alaingdl opened this issue Aug 2, 2023 · 3 comments
Closed

openmp & OSX 13.5 #1609

alaingdl opened this issue Aug 2, 2023 · 3 comments
Labels
needs more information! Further information is requested

Comments

@alaingdl
Copy link
Contributor

alaingdl commented Aug 2, 2023

Hi

The script to compile GDL on OSX 13.5 does work perfectly, on a machine with 12 cores, and on my old laptop with 4 cores and U2204.
But after that, performances issues : time to run test_all_basic_functions is about 8 times slower on Mac than on my Linux. I suspect OpenMP is not activated on OSX (but the cope number is detected in !CPU). Any hope for improvement in the way to really trun on OpenMP with Clang on OSX ?

Mac (12 cores) % Time elapsed ALL TESTS:       0.91960096 seconds.
Linux (4 cores) % Time elapsed ALL TESTS:       0.16531396 seconds.
@GillesDuvert
Copy link
Contributor

Nothing really new, eh?
See #1498 (comment)
by linking with the library provided by R folks, openmp is enabled.

@GillesDuvert GillesDuvert added the needs more information! Further information is requested label Aug 11, 2023
@GillesDuvert
Copy link
Contributor

As of today, the DMG (valid for Intel 64 bits Macs) prepared by a (weekly) release (so, compiled with optimization) seems to have all libraries inside and thus can be installed on any mac without need of Brew, compile etc, and actually runs well.
-> This has to be tested, I've only a MacOS Catalina 10.15.7 in a VM for devel and test, pity me.
-> But parallellism is included, as setting CPU,TPOOL_NTHREADS=1 and CPU,TPOOL_NTHREADS=4 display accelerated times in the latter case. Of course in a VM, probably not exciting times.
-> #1498 on the contary show that on M1, more work is needed.

As this eases terribly the maintainance, I plan to remove all explanations on how to install GDL on Mac in favor of just installing the DMG, so it is important to test it in real life.

@GillesDuvert
Copy link
Contributor

please follow this discussion on #1675

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs more information! Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants