radeonsi: don't set prefer_mono for fetched instance divisors
authorMarek Olšák <marek.olsak@amd.com>
Tue, 10 Aug 2021 08:35:04 +0000 (04:35 -0400)
committerMarge Bot <eric+marge@anholt.net>
Fri, 10 Sep 2021 23:32:02 +0000 (23:32 +0000)
It's not necessary because the overhead is very low and the comment isn't
true anymore. (the divisions are fast now)

Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12812>

src/gallium/drivers/radeonsi/si_state_shaders.c

index cd8387f..5ee2711 100644 (file)
@@ -1779,11 +1779,6 @@ void si_shader_selector_key_vs(struct si_context *sctx, struct si_shader_selecto
    prolog_key->instance_divisor_is_one = elts->instance_divisor_is_one;
    prolog_key->instance_divisor_is_fetched = elts->instance_divisor_is_fetched;
 
-   /* Prefer a monolithic shader to allow scheduling divisions around
-    * VBO loads. */
-   if (prolog_key->instance_divisor_is_fetched)
-      key->opt.prefer_mono = 1;
-
    unsigned count = MIN2(vs->info.num_inputs, elts->count);
    unsigned count_mask = (1 << count) - 1;
    unsigned fix = elts->fix_fetch_always & count_mask;