Skip to content

Commit

Permalink
The Barrier Pass does not need to split kernels at subgroup barriers (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
AmyCodeplay authored Aug 28, 2023
1 parent bf7c3f7 commit 89defaa
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion modules/compiler/utils/include/compiler/utils/builtin_info.h
Original file line number Diff line number Diff line change
Expand Up @@ -444,7 +444,9 @@ class BuiltinInfo {
/// @brief Returns true if the mux builtin has a barrier ID as its first
/// operand.
static bool isMuxBuiltinWithBarrierID(BuiltinID ID) {
if (isMuxControlBarrierID(ID)) {
// NOTE sub group barriers do have an ID operand, but the handle barriers
// pass doesn't need to know about that.
if (ID == eMuxBuiltinWorkGroupBarrier) {
return true;
}
auto Info = isMuxGroupCollective(ID);
Expand Down

0 comments on commit 89defaa

Please sign in to comment.