virgl: Always make some extra temps available for transformations
authorGert Wollny <gert.wollny@collabora.com>
Sat, 9 Apr 2022 09:54:29 +0000 (11:54 +0200)
committerMarge Bot <emma+marge@anholt.net>
Tue, 12 Apr 2022 10:44:17 +0000 (10:44 +0000)
The host driver will optimize unused variables away, and checking thoroughly whether we
may need an extra temp is just uselessly costly.

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Emma Anholt <emma@anholt.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15836>

src/gallium/drivers/virgl/virgl_tgsi.c

index f14583a..2a4f8bd 100644 (file)
@@ -189,11 +189,9 @@ virgl_tgsi_transform_prolog(struct tgsi_transform_context * ctx)
 {
    struct virgl_transform_context *vtctx = (struct virgl_transform_context *)ctx;
 
-   if (vtctx->info.uses_doubles || vtctx->info.file_count[TGSI_FILE_SAMPLER_VIEW]) {
-      vtctx->src_temp = vtctx->next_temp;
-      vtctx->next_temp += 4;
-      tgsi_transform_temps_decl(ctx, vtctx->src_temp, vtctx->src_temp + 3);
-   }
+   vtctx->src_temp = vtctx->next_temp;
+   vtctx->next_temp += 4;
+   tgsi_transform_temps_decl(ctx, vtctx->src_temp, vtctx->src_temp + 3);
 
    if (vtctx->num_writemask_fixups) {
       vtctx->writemask_fixup_temps = vtctx->next_temp;