Skip to content

Commit 35bc351

Browse files
committed
Cleanup OMPT after merge
1 parent 29583da commit 35bc351

File tree

5 files changed

+5
-19
lines changed

5 files changed

+5
-19
lines changed

runtime/CMakeLists.txt

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -194,12 +194,6 @@ elseif("${libomp_build_type_lowercase}" STREQUAL "minsizerel")
194194
set(MINSIZEREL_BUILD TRUE)
195195
endif()
196196

197-
# OMPD-support
198-
set(OMPD_SUPPORT FALSE)
199-
if("${ompd_support}") # string "on" or "ON" is seen as boolean TRUE
200-
set(OMPD_SUPPORT TRUE)
201-
endif()
202-
203197
# Include itt notify interface?
204198
set(LIBOMP_USE_ITT_NOTIFY TRUE CACHE BOOL
205199
"Enable ITT notify?")
@@ -314,6 +308,9 @@ endif()
314308
set(LIBOMP_OMPT_SUPPORT ${OMPT_DEFAULT} CACHE BOOL
315309
"OMPT-support?")
316310

311+
set(LIBOMP_OMPD_SUPPORT FALSE CACHE BOOL "OMPD-support?")
312+
313+
317314
set(LIBOMP_OMPT_DEBUG FALSE CACHE BOOL
318315
"Trace OMPT initialization?")
319316
set(LIBOMP_OMPT_OPTIONAL TRUE CACHE BOOL

runtime/src/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ if(WIN32)
194194
libomp_append(LIBOMP_MASM_DEFINITIONS "-D_M_IA32" IF_TRUE IA32)
195195
libomp_append(LIBOMP_MASM_DEFINITIONS "-D_M_AMD64" IF_TRUE INTEL64)
196196
libomp_append(LIBOMP_MASM_DEFINITIONS "-DOMPT_SUPPORT" IF_TRUE_1_0 LIBOMP_OMPT_SUPPORT)
197-
libomp_append(LIBOMP_MASM_DEFINITIONS "OMPD_SUPPORT" IF_TRUE_1_0 LIBOMP_OMPD_SUPPORT)
197+
libomp_append(LIBOMP_MASM_DEFINITIONS "-DOMPD_SUPPORT" IF_TRUE_1_0 LIBOMP_OMPD_SUPPORT)
198198
libomp_list_to_string("${LIBOMP_MASM_DEFINITIONS}" LIBOMP_MASM_DEFINITIONS)
199199
set_property(SOURCE z_Windows_NT-586_asm.asm APPEND_STRING PROPERTY COMPILE_FLAGS " ${LIBOMP_MASM_DEFINITIONS}")
200200
set_source_files_properties(thirdparty/ittnotify/ittnotify_static.c PROPERTIES COMPILE_DEFINITIONS "UNICODE")

runtime/src/kmp_runtime.cpp

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1792,14 +1792,11 @@ int __kmp_fork_call(ident_t *loc, int gtid,
17921792
exit_runtime_p
17931793
#endif
17941794
);
1795-
#if OMPT_SUPPORT
1796-
*exit_runtime_p=0;
1797-
#endif
17981795
}
17991796

18001797
#if OMPT_SUPPORT
18011798
if (ompt_enabled.enabled) {
1802-
exit_runtime_p = NULL;
1799+
*exit_runtime_p = NULL;
18031800
if (ompt_enabled.ompt_callback_implicit_task) {
18041801
ompt_callbacks.ompt_callback(ompt_callback_implicit_task)(
18051802
ompt_scope_end, NULL, &(task_info->task_data), 1,
@@ -1897,9 +1894,6 @@ int __kmp_fork_call(ident_t *loc, int gtid,
18971894
exit_runtime_p
18981895
#endif
18991896
);
1900-
#if OMPT_SUPPORT
1901-
*exit_runtime_p=0;
1902-
#endif
19031897
}
19041898

19051899
#if OMPT_SUPPORT

runtime/src/kmp_tasking.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -587,9 +587,6 @@ void __kmpc_omp_task_begin_if0(ident_t *loc_ref, kmp_int32 gtid,
587587
OMPT_LOAD_RETURN_ADDRESS(gtid));
588588
return;
589589
}
590-
if (ompt_enabled)
591-
taskdata->ompt_task_info.scheduling_parent = current_task;
592-
593590
#endif
594591
__kmpc_omp_task_begin_if0_template<false>(loc_ref, gtid, task, NULL, NULL);
595592
}

runtime/src/ompt-internal.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@ typedef struct ompt_callbacks_active_s {
3939
(info->td_flags.final ? ompt_task_final : 0x0) | \
4040
(info->td_flags.merged_if0 ? ompt_task_mergeable : 0x0)
4141

42-
//typedef struct kmp_taskdata kmp_taskdata_t;
43-
4442
typedef struct {
4543
ompt_frame_t frame;
4644
ompt_data_t task_data;

0 commit comments

Comments
 (0)