From a2748c74d68bd3748dcf626d828ee50a19cd1324 Mon Sep 17 00:00:00 2001 From: Jonas Hahnfeld Date: Mon, 22 Jul 2019 18:46:02 +0000 Subject: [OMPT] Cleanup reset of exit_frame pointer This is done at call-site and does not need to be handled in __kmp_invoke_microtask. It was already absent from the x86 and x86_64 assembly, this patch removes it from the generic implementation in z_Linux_util.cpp and adds documentation for AArch64 and PPC64 that it's actually not needed. I can't test on these architectures, so I don't want to change the code just because it looks right :) While at it, rename some variables for consistency and add a check in test/ompt/parallel/normal.c that the pointer was reset before entering the barrier. Differential Revision: https://reviews.llvm.org/D64442 llvm-svn: 366721 --- openmp/runtime/src/z_Linux_util.cpp | 4 ---- 1 file changed, 4 deletions(-) (limited to 'openmp/runtime/src/z_Linux_util.cpp') diff --git a/openmp/runtime/src/z_Linux_util.cpp b/openmp/runtime/src/z_Linux_util.cpp index 1983fc2b983..5f80c723a62 100644 --- a/openmp/runtime/src/z_Linux_util.cpp +++ b/openmp/runtime/src/z_Linux_util.cpp @@ -2415,10 +2415,6 @@ int __kmp_invoke_microtask(microtask_t pkfn, int gtid, int tid, int argc, break; } -#if OMPT_SUPPORT - *exit_frame_ptr = 0; -#endif - return 1; } -- cgit v1.2.3