From bcd8cbe9a529e9d2de73b98a36adaecf28eeffc9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Wed, 12 Jul 2023 23:09:16 -0400 Subject: [PATCH] Revert "ac/nir/ngg: Follow intrinsic sources when analyzing before culling." This reverts commit 411f69b9c5b884f6802758347e72a10ec5564df1. It broke tessellation in Unigine Heaven with radeonsi. Fixes: 411f69b9c5b884 - ac/nir/ngg: Follow intrinsic sources when analyzing before culling. Part-of: (cherry picked from commit 043dcfad04d89c5b68a82f0c68638f38a85fdb05) --- .pick_status.json | 2 +- src/amd/common/ac_nir_lower_ngg.c | 5 ----- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/.pick_status.json b/.pick_status.json index df6e523c96f..98dcaa4398a 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -64,7 +64,7 @@ "description": "Revert \"ac/nir/ngg: Follow intrinsic sources when analyzing before culling.\"", "nominated": true, "nomination_type": 1, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": "411f69b9c5b884f6802758347e72a10ec5564df1", "notes": null diff --git a/src/amd/common/ac_nir_lower_ngg.c b/src/amd/common/ac_nir_lower_ngg.c index d32dae39e90..96e6367aa5f 100644 --- a/src/amd/common/ac_nir_lower_ngg.c +++ b/src/amd/common/ac_nir_lower_ngg.c @@ -1075,11 +1075,6 @@ analyze_shader_before_culling_walk(nir_ssa_def *ssa, break; } - const unsigned num_srcs = nir_intrinsic_infos[intrin->intrinsic].num_srcs; - for (unsigned i = 0; i < num_srcs; ++i) { - analyze_shader_before_culling_walk(intrin->src[i].ssa, flag, s); - } - break; } case nir_instr_type_alu: {