-
Notifications
You must be signed in to change notification settings - Fork 45
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
tech-gpu and v5.3 not compiling on LUMI with OpenMP-GPU enabled #180
Comments
Thanks Nicola for spotting this. This looks a big bug to me, since we call these interfaces without declaring them through the whole source. I'll restore the mentioned line asap in 5.3. |
MODIFIED * configure include/headers/common/y_memory.h include/version/version.m4 Bugs: - fix for issue #180 Patch sent by: Davide Sangalli <davide.sangalli@gmail.com>
Dear all, the release should remain stable up to when we all merge with it. If the code moves the merging becomes troublesome. We should discuss how to handle the validation period after a release has been proposed. Maybe new releases (5.3-alpha, 5.3-beta....) or (5.3.1, 5.3.2....). |
The second issue after the restoring of the EDIT: this is not true, the |
Fixed with commit 04fe5ae |
The branch tech-gpu is not anymore compilable on LUMI-G (with OpenMP-GPU enabled) after alignment with version 5.3:
The issue is caused by the remove of these three lines in the file
include/headers/common/y_memory.h
that is included in all the source files that allocate or deallocate memory:However, adding again these lines causes another issue:
The text was updated successfully, but these errors were encountered: