From 9329da6d88bf33d95396fcf3dc113c17d577a861 Mon Sep 17 00:00:00 2001 From: Caio Oliveira Date: Mon, 6 Apr 2026 13:12:35 -0700 Subject: [PATCH] brw: Don't set saturate for SYNC instruction This helper might be used as by another instruction emission, which itself might have set the saturate bit in the default state. This might result in the SYNC being created already with saturate bit set. Since SYNC doesn't have saturate, clear that field instead of sometimes having it set. Reviewed-by: Paulo Zanoni Reviewed-by: Ian Romanick Part-of: --- src/intel/compiler/brw/brw_eu_emit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/compiler/brw/brw_eu_emit.c b/src/intel/compiler/brw/brw_eu_emit.c index 364fa667716..cd1234b0825 100644 --- a/src/intel/compiler/brw/brw_eu_emit.c +++ b/src/intel/compiler/brw/brw_eu_emit.c @@ -890,6 +890,7 @@ void brw_NOP(struct brw_codegen *p) void brw_SYNC(struct brw_codegen *p, enum tgl_sync_function func) { brw_eu_inst *insn = next_insn(p, BRW_OPCODE_SYNC); + brw_eu_inst_set_saturate(p->devinfo, insn, 0); brw_eu_inst_set_cond_modifier(p->devinfo, insn, func); }