Page MenuHomePhabricator

[OpenMP] Introduce aligned synchronization into the new device RT
ClosedPublic

Authored by jdoerfert on Oct 20 2021, 9:29 AM.

Details

Summary

We will later use the fact that a barrier is aligned to reason about
thread divergence. For now we introduce the assumption and some more
documentation.

Diff Detail

Event Timeline

jdoerfert created this revision.Oct 20 2021, 9:29 AM
jdoerfert requested review of this revision.Oct 20 2021, 9:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 20 2021, 9:29 AM
Herald added a subscriber: sstefan1. · View Herald Transcript
tianshilei1992 added inline comments.Oct 20 2021, 9:54 AM
openmp/libomptarget/DeviceRTL/src/Synchronization.cpp
186

We don't have to use a named barrier for aligned barrier, do we?

jdoerfert added inline comments.Oct 20 2021, 3:35 PM
openmp/libomptarget/DeviceRTL/src/Synchronization.cpp
186

I guess not.

jdoerfert updated this revision to Diff 381121.Oct 20 2021, 5:30 PM

use syncthreads

JonChesterfield added inline comments.
openmp/libomptarget/DeviceRTL/src/Parallelism.cpp
96–98

Thank you for the comments here

This revision is now accepted and ready to land.Oct 22 2021, 11:41 AM