diff options
author | Jonas Hahnfeld <hahnjo@hahnjo.de> | 2019-07-22 18:46:02 +0000 |
---|---|---|
committer | Jonas Hahnfeld <hahnjo@hahnjo.de> | 2019-07-22 18:46:02 +0000 |
commit | a2748c74d68bd3748dcf626d828ee50a19cd1324 (patch) | |
tree | 706ff54cc15fc7da500f0b7f9144faf0253d36b0 /openmp/runtime/src/z_Linux_util.cpp | |
parent | 06b8fe8d030e739367327093768c36e0e9dd2217 (diff) | |
download | bcm5719-llvm-a2748c74d68bd3748dcf626d828ee50a19cd1324.tar.gz bcm5719-llvm-a2748c74d68bd3748dcf626d828ee50a19cd1324.zip |
[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
Diffstat (limited to 'openmp/runtime/src/z_Linux_util.cpp')
-rw-r--r-- | openmp/runtime/src/z_Linux_util.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
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; } |