microsoft/compiler: Only set typed UAV load feature bit for multi-comp loads
authorJesse Natalie <jenatali@microsoft.com>
Wed, 15 Mar 2023 16:58:10 +0000 (09:58 -0700)
committerMarge Bot <emma+marge@anholt.net>
Thu, 23 Mar 2023 21:48:43 +0000 (21:48 +0000)
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21913>

src/microsoft/compiler/nir_to_dxil.c

index ce2f3fa..4d5a809 100644 (file)
@@ -4056,12 +4056,8 @@ emit_image_load(struct ntd_context *ctx, nir_intrinsic_instr *intr)
       store_dest(ctx, &intr->dest, i, component, out_type);
    }
 
-   /* FIXME: This flag should be set to true when the RWTexture is attached
-    * a vector, and we always declare a vec4 right now, so it should always be
-    * true. Might be worth reworking the dxil_module_get_res_type() to use a
-    * scalar when the image only has one component.
-    */
-   ctx->mod.feats.typed_uav_load_additional_formats = true;
+   if (num_components > 1)
+      ctx->mod.feats.typed_uav_load_additional_formats = true;
 
    return true;
 }