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

Initial support for Cray OpenMP offloading on AMD GPUs #971

Open
wants to merge 73 commits into
base: atmosphere/develop-openacc
Choose a base branch
from

Conversation

jychang48
Copy link

Currently only supported through the cray ftn compiler.
All OMP directives added on top of existing ACC directives.
Only core dynamics ported. Physics not touched.
Macro definitions (MPAS_AMD_OFFLOAD, GPUOMP, etc) are temporary measures and will be renamed or removed when the AOMP compiler is supported. Should also eventually work with Intel and NVIDIA GPUs.

Justin Chang and others added 30 commits January 11, 2022 13:35
…ys_XXX.F

At this point, all of these ACC directives are ported to OpenMP offloading:

!$acc declare create(...)
!$acc data create(...)
!$acc data copyin(...)
!$acc enter data create(...)
!$acc enter data copyin(...)
jychang48 and others added 25 commits February 7, 2022 21:12
atm_divergence_damping_3d
atm_compute_dyn_tend_work
atm_compute_dyn_tend_work_rk23
atm_advance_acoustic_step_work
physics_get_tend
…tnpass

Atmosphere/develop openacc amd ftnpass
@mgduda mgduda self-requested a review June 4, 2022 03:05
@mgduda mgduda requested a review from gdicker1 June 13, 2022 23:04
@jychang48
Copy link
Author

Note, many of the changes in this PR dealt with manually incorporating !$omp target enter data map ... for all data structures involving !$acc declare create due to the limitations of the compiler at the time. It is my hope today that one can simply use !$omp declare target for such data now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants