Skip to content

Commit

Permalink
Some more updates for NEMOv5
Browse files Browse the repository at this point in the history
  • Loading branch information
sergisiso committed Dec 17, 2024
1 parent 0f44e28 commit 873be3b
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 2 deletions.
3 changes: 3 additions & 0 deletions examples/nemo/scripts/omp_cpu_trans.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,9 @@
# List of all files that psyclone will skip processing
FILES_TO_SKIP = PASSTHROUGH_ISSUES

if PROFILING_ENABLED:
# Fails with profiling enabled. issue #2723
FILES_TO_SKIP.append("mppini.f90")

def trans(psyir):
''' Add OpenMP Parallel and Do directives to all loops, including the
Expand Down
8 changes: 8 additions & 0 deletions examples/nemo/scripts/omp_gpu_trans.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@
# Illegal address during kernel execution
"dynzdf.f90", # returned error 700 (CUDA_ERROR_ILLEGAL_ADDRESS)
"traatf_qco.f90", # Runtime: Failed to find device function
"lbclnk.f90", # Improve performance until #2751
]

PRIVATISATION_ISSUES = [
Expand Down Expand Up @@ -115,6 +116,13 @@ def trans(psyir):

for subroutine in psyir.walk(Routine):

# Skip things from the initialisation
if (subroutine.name.endswith('_alloc') or
subroutine.name.endswith('_init') or
subroutine.name.startswith('Agrif') or
subroutine.name == 'dom_msk'):
continue

if PROFILING_ENABLED:
add_profiling(subroutine.children)

Expand Down
3 changes: 1 addition & 2 deletions examples/nemo/scripts/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,7 @@
]

# If routine names contain these substrings then we do not profile them
PROFILING_IGNORE = ["_init", "_rst", "alloc", "agrif", "flo_dom",
"macho", "mpp_", "nemo_gcm",
PROFILING_IGNORE = ["flo_dom", "macho", "mpp_", "nemo_gcm",
# These are small functions that the addition of profiling
# prevents from being in-lined (and then breaks any attempt
# to create OpenACC regions with calls to them)
Expand Down

0 comments on commit 873be3b

Please sign in to comment.