diff --git a/.pick_status.json b/.pick_status.json index d9d6b809f9b..e5272e82251 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -1484,7 +1484,7 @@ "description": "brw: drop buggy SLM optimization", "nominated": true, "nomination_type": 1, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": null, "notes": null diff --git a/src/intel/compiler/brw/brw_from_nir.cpp b/src/intel/compiler/brw/brw_from_nir.cpp index d402460f976..8c0cd168f95 100644 --- a/src/intel/compiler/brw/brw_from_nir.cpp +++ b/src/intel/compiler/brw/brw_from_nir.cpp @@ -6033,12 +6033,6 @@ brw_from_nir_emit_intrinsic(nir_to_brw_state &ntb, if (opcode == BRW_OPCODE_NOP) break; - if (s.nir->info.shared_size > 0) { - assert(mesa_shader_stage_uses_workgroup(s.stage)); - } else { - slm_fence = false; - } - /* If the workgroup fits in a single HW thread, the messages for SLM are * processed in-order and the shader itself is already synchronized so * the memory fence is not necessary.