Skip to content

Commit b5ce41c

Browse files
committed
Small refactoring change.
1 parent aa7664f commit b5ce41c

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

kernel_tuner/utils/directives.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ def wrap_timing_fortran(code: str) -> str:
369369

370370
def end_timing_cxx(code: str) -> str:
371371
"""In C++ we need to return the measured time"""
372-
return code + "\nreturn elapsed_time.count();\n"
372+
return "\n".join([code, "return elapsed_time.count();\n"])
373373

374374

375375
def wrap_data(code: str, langs: Code, data: dict, preprocessor: list = None, user_dimensions: dict = None) -> str:
@@ -385,7 +385,7 @@ def wrap_data(code: str, langs: Code, data: dict, preprocessor: list = None, use
385385
elif is_openacc(langs.directive) and is_fortran(langs.language):
386386
intro += create_data_directive_openacc_fortran(name, size)
387387
outro += exit_data_directive_openacc_fortran(name, size)
388-
return intro + code + outro
388+
return "\n".join([intro, code, outro])
389389

390390

391391
def extract_directive_code(code: str, langs: Code, kernel_name: str = None) -> dict:

test/utils/test_directives.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,13 +105,13 @@ def test_wrap_data():
105105
code_f90 = "! this is a comment\n"
106106
data = {"array": ["int*", "size"]}
107107
preprocessor = ["#define size 42"]
108-
expected_cxx = "#pragma acc enter data create(array[:42])\n#pragma acc update device(array[:42])\n// this is a comment\n#pragma acc exit data copyout(array[:42])\n"
108+
expected_cxx = "#pragma acc enter data create(array[:42])\n#pragma acc update device(array[:42])\n\n// this is a comment\n\n#pragma acc exit data copyout(array[:42])\n"
109109
assert wrap_data(code_cxx, acc_cxx, data, preprocessor, None) == expected_cxx
110-
expected_f90 = "!$acc enter data create(array(:42))\n!$acc update device(array(:42))\n! this is a comment\n!$acc exit data copyout(array(:42))\n"
110+
expected_f90 = "!$acc enter data create(array(:42))\n!$acc update device(array(:42))\n\n! this is a comment\n\n!$acc exit data copyout(array(:42))\n"
111111
assert wrap_data(code_f90, acc_f90, data, preprocessor, None) == expected_f90
112112
data = {"matrix": ["float*", "rows,cols"]}
113113
preprocessor = ["#define rows 42", "#define cols 84"]
114-
expected_f90 = "!$acc enter data create(matrix(:42,:84))\n!$acc update device(matrix(:42,:84))\n! this is a comment\n!$acc exit data copyout(matrix(:42,:84))\n"
114+
expected_f90 = "!$acc enter data create(matrix(:42,:84))\n!$acc update device(matrix(:42,:84))\n\n! this is a comment\n\n!$acc exit data copyout(matrix(:42,:84))\n"
115115
assert wrap_data(code_f90, acc_f90, data, preprocessor, None) == expected_f90
116116
dimensions = {"rows": 42, "cols": 84}
117117
assert wrap_data(code_f90, acc_f90, data, user_dimensions=dimensions) == expected_f90

0 commit comments

Comments
 (0)