From b16c169b6dbca379da25f266d9c177c573d3ffa9 Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Tue, 24 Aug 2021 10:43:08 -0700 Subject: [PATCH] vc4: Don't call nir_lower_flrp in vc4_optimize_nir It's already called by st_nir_opts, so it shouldn't be necessary to do it again here. This is only compile tested. I have not collected any shader-db or fossil-db data. Acked-by: Juan A. Suarez Romero Reviewed-by: Alyssa Rosenzweig Part-of: --- src/gallium/drivers/vc4/vc4_program.c | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/src/gallium/drivers/vc4/vc4_program.c b/src/gallium/drivers/vc4/vc4_program.c index 1cfa7a884e7..f48240ea551 100644 --- a/src/gallium/drivers/vc4/vc4_program.c +++ b/src/gallium/drivers/vc4/vc4_program.c @@ -1486,10 +1486,6 @@ static void vc4_optimize_nir(struct nir_shader *s) { bool progress; - unsigned lower_flrp = - (s->options->lower_flrp16 ? 16 : 0) | - (s->options->lower_flrp32 ? 32 : 0) | - (s->options->lower_flrp64 ? 64 : 0); do { progress = false; @@ -1511,16 +1507,6 @@ vc4_optimize_nir(struct nir_shader *s) NIR_PASS(progress, s, nir_opt_peephole_select, &peephole_select_options); NIR_PASS(progress, s, nir_opt_algebraic); NIR_PASS(progress, s, nir_opt_constant_folding); - if (lower_flrp != 0) { - NIR_PASS(progress, s, nir_lower_flrp, - lower_flrp, false /* always_precise */); - - /* Nothing should rematerialize any flrps, so we only - * need to do this lowering once. - */ - lower_flrp = 0; - } - NIR_PASS(progress, s, nir_opt_undef); NIR_PASS(progress, s, nir_opt_loop_unroll); } while (progress);