[Openmp-commits] [openmp] [libomptarget][nextgen-plugin] Add missing mutex lock (PR #83455)

via Openmp-commits openmp-commits at lists.llvm.org
Mon Mar 4 11:58:51 PST 2024


https://github.com/dhruvachak commented:

In the caller of waitOnStreamOperation, there is below locking. Can you explain why we still need additional locking?

----------------
Error AMDGPUStreamTy::waitEvent(const AMDGPUEventTy &Event) {
  // Retrieve the recorded stream on the event.
  AMDGPUStreamTy &RecordedStream = *Event.RecordedStream;
    
  std::scoped_lock<std::mutex, std::mutex> Lock(Mutex, RecordedStream.Mutex);


https://github.com/llvm/llvm-project/pull/83455


More information about the Openmp-commits mailing list