[Openmp-commits] [PATCH] D112861: [OpenMP][DeviceRTL] Fixed an issue that causes hang in SU3
Ye Luo via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Sat Oct 30 13:22:20 PDT 2021
ye-luo added inline comments.
Comment at: openmp/libomptarget/DeviceRTL/src/Parallelism.cpp:129
+ // __kmpc_target_deinit may not hold.
> ye-luo wrote:
> > This explanation seems plausible. Are the synchronization requirement of ASSERT and why being documented somewhere?
> Have no idea.
synchronize::threadsAligned() has been called at the end of above scope.
What caused the out of sync and needs this additional call?
Why ASSERT requires synchronization.
All needs explanation or documentation.
"Synchronize all threads to make sure every thread exits the scope above;"
Just sounds plausible
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the Openmp-commits