Skip to content

Commit ebbd43d

Browse files
committed
Check different exclusions
1 parent df12ff6 commit ebbd43d

File tree

3 files changed

+21
-14
lines changed

3 files changed

+21
-14
lines changed

examples/nemo/scripts/omp_cpu_trans.py

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -69,19 +69,19 @@
6969
'par_kind.F90',
7070
]
7171

72-
if not NEMOV4:
73-
FILES_TO_SKIP.extend([
74-
# TODO #3112: These produce diverging run.stat results in NEMOv5 BENCH
75-
"dynhpg.f90",
76-
"dynspg_ts.f90",
77-
"icedyn_rhg_evp.f90",
78-
"icethd_dh.f90",
79-
"icevar.f90",
80-
"iom_nf90.f90",
81-
"sbcssm.f90",
82-
"tramle.f90",
83-
"trazdf.f90",
84-
])
72+
# if not NEMOV4:
73+
# FILES_TO_SKIP.extend([
74+
# # TODO #3112: These produce diverging run.stat results in NEMOv5 BENCH
75+
# "dynhpg.f90",
76+
# "dynspg_ts.f90",
77+
# "icedyn_rhg_evp.f90",
78+
# "icethd_dh.f90",
79+
# "icevar.f90",
80+
# "iom_nf90.f90",
81+
# "sbcssm.f90",
82+
# "tramle.f90",
83+
# "trazdf.f90",
84+
# ])
8585

8686
if PROFILING_ENABLED:
8787
# Fails with profiling enabled. issue #2723

examples/nemo/scripts/omp_gpu_trans.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,11 @@
8181

8282
NEMOV5_EXCLUSIONS = []
8383

84+
NEMOV4_EXCLUSIONS = [
85+
"dynspg_ts.f90",
86+
# "tranxt.f90",
87+
]
88+
8489
SKIP_FOR_PERFORMANCE = [
8590
"iom.f90",
8691
"iom_nf90.f90",
@@ -167,6 +172,8 @@ def trans(psyir):
167172
# insert profiling hooks later on.
168173
if psyir.name in SKIP_FOR_PERFORMANCE:
169174
continue
175+
if NEMOV4 and psyir.name in NEMOV4_EXCLUSIONS:
176+
continue
170177
if not NEMOV4 and psyir.name in NEMOV5_EXCLUSIONS:
171178
continue
172179
# ICE routines do not perform well on GPU, so we skip them

examples/nemo/scripts/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,7 @@ def insert_explicit_loop_parallelism(
423423
424424
'''
425425
nemo_v4 = os.environ.get('NEMOV4', False)
426-
# These are both in "dynstg_ts.f90" and has a big performance impact
426+
# These are both in "dynspg_ts.f90" and has a big performance impact
427427
if schedule.name in ("ts_wgt", "ts_rst"):
428428
return # TODO #2937 WaW dependency incorrectly considered private
429429
# Add the parallel directives in each loop

0 commit comments

Comments
 (0)