diff --git a/.pick_status.json b/.pick_status.json index a20a2a9bcba..290612b93a4 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -4794,7 +4794,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 4667600bc4c..7970eba3a21 100644 --- a/src/intel/compiler/brw/brw_from_nir.cpp +++ b/src/intel/compiler/brw/brw_from_nir.cpp @@ -5158,12 +5158,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.