tgsi: Don't bother setting the TGSI_PROPERTY_SEPARABLE_PROGRAM
authorGert Wollny <gert.wollny@collabora.com>
Sat, 2 Jul 2022 08:02:16 +0000 (10:02 +0200)
committerMarge Bot <emma+marge@anholt.net>
Thu, 7 Jul 2022 09:46:53 +0000 (09:46 +0000)
Most drivers don't care about the property, and virgl should only handle
it if the host supports it.

This is a partial revert of b634030, i.e. we keep the definition of the
property, but we don't set it only based on the shader info.

Fixes: b63403054237c3e408f2662beb52ec65db0aee30
  tgsi: Add SEPARABLE_PROGRAM property

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17344>

src/gallium/auxiliary/tgsi/tgsi_ureg.c

index d3ba3f0..398d59b 100644 (file)
@@ -2408,9 +2408,6 @@ ureg_setup_shader_info(struct ureg_program *ureg,
    if (info->layer_viewport_relative)
       ureg_property(ureg, TGSI_PROPERTY_LAYER_VIEWPORT_RELATIVE, 1);
 
-   if (info->separate_shader)
-      ureg_property(ureg, TGSI_PROPERTY_SEPARABLE_PROGRAM, 1);
-
    switch (info->stage) {
    case MESA_SHADER_VERTEX:
       ureg_setup_clipdist_info(ureg, info);