From 9ce869aef571f30bb5dab0e35c57900bcf6fab56 Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Wed, 9 Apr 2025 19:52:47 -0700 Subject: [PATCH] brw/cmod: Delete some stale comment text MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Stale like the mummified remains of Ötzi, The Iceman. Reviewed-by: Matt Turner Part-of: --- src/intel/compiler/brw_opt_cmod_propagation.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/intel/compiler/brw_opt_cmod_propagation.cpp b/src/intel/compiler/brw_opt_cmod_propagation.cpp index 55365f317d0..6cd352669b2 100644 --- a/src/intel/compiler/brw_opt_cmod_propagation.cpp +++ b/src/intel/compiler/brw_opt_cmod_propagation.cpp @@ -242,12 +242,10 @@ opt_cmod_propagation_local(const intel_device_info *devinfo, bblock_t *block) (inst->opcode != BRW_OPCODE_CMP || inst->src[1].is_zero())) continue; - /* Only an AND.NZ can be propagated. Many AND.Z instructions are - * generated (for ir_unop_not in brw_shader::emit_bool_to_cond_code). - * Propagating those would require inverting the condition on the CMP. - * This changes both the flag value and the register destination of the - * CMP. That result may be used elsewhere, so we can't change its value - * on a whim. + /* Only an AND.NZ can be propagated. Propagating AND.Z would require + * inverting the condition on the CMP. This changes both the flag value + * and the register destination of the CMP. That result may be used + * elsewhere, so we can't change its value on a whim. */ if (inst->opcode == BRW_OPCODE_AND && !(inst->src[1].is_one() &&