Re: [Mesa-dev] [PATCH 18/92] gallium: add PIPE_CAP_NIR_LOWER_IO

Nicolai Hähnle Tue, 27 Jun 2017 00:43:55 -0700

On 27.06.2017 02:25, Connor Abbott wrote:

Ok, so I just looked into it a little further, and I guess that since
nir_to_llvm needs to know about the ABI to know where all the
"special" inputs/outputs like tess factors, position etc. are,
switching radv to using the normal input/output intrinsics would be
better done as part of your series, so I'll leave that to you. In the
meantime, though, I'm going to send out some patches which improve
radv's local variable handling which also remove one of the users of
radv_get_deref_offset() that ultimately needs to go. It'll cause some
rebase conflicts for you, but they shouldn't be so hard to solve.


Sure, no problem.

I remember looking at this and not wanting to change it mostly because Ididn't want to perturb the radv path too much. I'll take a look at it again.


Cheers,
Nicolai


On Mon, Jun 26, 2017 at 12:25 PM, Connor Abbott <cwabbo...@gmail.com> wrote:

Forgot to mention... I think I'll take a pass at doing this today.
It'll distract me from some weird bug I've been trying to fight with
the AMD_shader_ballot stuff :)

On Mon, Jun 26, 2017 at 12:24 PM, Connor Abbott <cwabbo...@gmail.com> wrote:

So, I think that rather than doing this, we should make radv call
nir_lower_io instead. There's currently a bunch of code in the
NIR-to-LLVM translation to calculate dereference offsets and split up
loads, which is just silly - use the lowering pass instead!

On Mon, Jun 26, 2017 at 7:09 AM, Nicolai Hähnle <nhaeh...@gmail.com> wrote:

From: Nicolai Hähnle <nicolai.haeh...@amd.com>

Existing NIR drivers return 1 here, since that reflects the current
behavior.
---
  src/gallium/docs/source/screen.rst               | 5 +++++
  src/gallium/drivers/etnaviv/etnaviv_screen.c     | 1 +
  src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
  src/gallium/drivers/i915/i915_screen.c           | 1 +
  src/gallium/drivers/llvmpipe/lp_screen.c         | 1 +
  src/gallium/drivers/nouveau/nv30/nv30_screen.c   | 1 +
  src/gallium/drivers/nouveau/nv50/nv50_screen.c   | 1 +
  src/gallium/drivers/nouveau/nvc0/nvc0_screen.c   | 1 +
  src/gallium/drivers/r300/r300_screen.c           | 1 +
  src/gallium/drivers/r600/r600_pipe.c             | 1 +
  src/gallium/drivers/radeonsi/si_pipe.c           | 1 +
  src/gallium/drivers/softpipe/sp_screen.c         | 1 +
  src/gallium/drivers/svga/svga_screen.c           | 1 +
  src/gallium/drivers/swr/swr_screen.cpp           | 1 +
  src/gallium/drivers/vc4/vc4_screen.c             | 1 +
  src/gallium/drivers/virgl/virgl_screen.c         | 1 +
  src/gallium/include/pipe/p_defines.h             | 1 +
  17 files changed, 21 insertions(+)

diff --git a/src/gallium/docs/source/screen.rst 
b/src/gallium/docs/source/screen.rst
index b375e53..9dab38b 100644
--- a/src/gallium/docs/source/screen.rst
+++ b/src/gallium/docs/source/screen.rst
@@ -391,20 +391,25 @@ The integer capabilities:
    shader outputs.
  * ``PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX``: Whether a buffer with just
    PIPE_BIND_CONSTANT_BUFFER can be legally passed to set_vertex_buffers.
  * ``PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION``: As the name says.
  * ``PIPE_CAP_POST_DEPTH_COVERAGE``: whether
    ``TGSI_PROPERTY_FS_POST_DEPTH_COVERAGE`` is supported.
  * ``PIPE_CAP_BINDLESS_TEXTURE``: Whether bindless texture operations are
    supported.
  * ``PIPE_CAP_NIR_SAMPLERS_AS_DEREF``: Whether NIR tex instructions should
    reference texture and sampler as NIR derefs instead of by indices.
+* ``PIPE_CAP_NIR_LOWER_IO``: Whether the nir_lower_io pass should be run to
+  replace variable accesses by the corresponding intrinsics. Note that when
+  this is false, the state tracker can no longer leave accesses to
+  default-block uniforms in the shader, meaning that nir_lower_uniforms_to_ubo
+  is used.


  .. _pipe_capf:

  PIPE_CAPF_*
  ^^^^^^^^^^^^^^^^

  The floating-point capabilities are:

  * ``PIPE_CAPF_MAX_LINE_WIDTH``: The maximum width of a regular line.
diff --git a/src/gallium/drivers/etnaviv/etnaviv_screen.c 
b/src/gallium/drivers/etnaviv/etnaviv_screen.c
index 85b7e91..7f2a231 100644
--- a/src/gallium/drivers/etnaviv/etnaviv_screen.c
+++ b/src/gallium/drivers/etnaviv/etnaviv_screen.c
@@ -252,20 +252,21 @@ etna_screen_get_param(struct pipe_screen *pscreen, enum 
pipe_cap param)
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;

     /* Stream output. */
     case PIPE_CAP_MAX_STREAM_OUTPUT_BUFFERS:
     case PIPE_CAP_STREAM_OUTPUT_PAUSE_RESUME:
     case PIPE_CAP_MAX_STREAM_OUTPUT_SEPARATE_COMPONENTS:
     case PIPE_CAP_MAX_STREAM_OUTPUT_INTERLEAVED_COMPONENTS:
        return 0;

     /* Geometry shader output, unsupported. */
diff --git a/src/gallium/drivers/freedreno/freedreno_screen.c 
b/src/gallium/drivers/freedreno/freedreno_screen.c
index 6380996..8fab5fd 100644
--- a/src/gallium/drivers/freedreno/freedreno_screen.c
+++ b/src/gallium/drivers/freedreno/freedreno_screen.c
@@ -174,20 +174,21 @@ fd_screen_get_param(struct pipe_screen *pscreen, enum 
pipe_cap param)
         case PIPE_CAP_TGSI_FS_COORD_PIXEL_CENTER_INTEGER:
         case PIPE_CAP_SEAMLESS_CUBE_MAP:
         case PIPE_CAP_VERTEX_COLOR_UNCLAMPED:
         case PIPE_CAP_QUADS_FOLLOW_PROVOKING_VERTEX_CONVENTION:
         case PIPE_CAP_VERTEX_BUFFER_OFFSET_4BYTE_ALIGNED_ONLY:
         case PIPE_CAP_VERTEX_BUFFER_STRIDE_4BYTE_ALIGNED_ONLY:
         case PIPE_CAP_VERTEX_ELEMENT_SRC_OFFSET_4BYTE_ALIGNED_ONLY:
         case PIPE_CAP_BUFFER_MAP_PERSISTENT_COHERENT:
         case PIPE_CAP_STRING_MARKER:
         case PIPE_CAP_MIXED_COLOR_DEPTH_BITS:
+       case PIPE_CAP_NIR_LOWER_IO:
                 return 1;

         case PIPE_CAP_VERTEXID_NOBASE:
                 return is_a3xx(screen) || is_a4xx(screen);

         case PIPE_CAP_USER_CONSTANT_BUFFERS:
                 return is_a4xx(screen) ? 0 : 1;

         case PIPE_CAP_COMPUTE:
                 return has_compute(screen);
diff --git a/src/gallium/drivers/i915/i915_screen.c 
b/src/gallium/drivers/i915/i915_screen.c
index 3a0dade..17f4858 100644
--- a/src/gallium/drivers/i915/i915_screen.c
+++ b/src/gallium/drivers/i915/i915_screen.c
@@ -304,20 +304,21 @@ i915_get_param(struct pipe_screen *screen, enum pipe_cap 
cap)
     case PIPE_CAP_INT64_DIVMOD:
     case PIPE_CAP_TGSI_TEX_TXF_LZ:
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;

     case PIPE_CAP_MAX_VIEWPORTS:
        return 1;

     case PIPE_CAP_MIN_MAP_BUFFER_ALIGNMENT:
        return 64;

     case PIPE_CAP_GLSL_FEATURE_LEVEL:
        return 120;
diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c 
b/src/gallium/drivers/llvmpipe/lp_screen.c
index 08edc08..df3edcf 100644
--- a/src/gallium/drivers/llvmpipe/lp_screen.c
+++ b/src/gallium/drivers/llvmpipe/lp_screen.c
@@ -350,20 +350,21 @@ llvmpipe_get_param(struct pipe_screen *screen, enum 
pipe_cap param)
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;
     }
     /* should only get here on unhandled cases */
     debug_printf("Unexpected PIPE_CAP %d query\n", param);
     return 0;
  }

  static int
  llvmpipe_get_shader_param(struct pipe_screen *screen,
                            enum pipe_shader_type shader,
diff --git a/src/gallium/drivers/nouveau/nv30/nv30_screen.c 
b/src/gallium/drivers/nouveau/nv30/nv30_screen.c
index 570a5d4..0acc12e 100644
--- a/src/gallium/drivers/nouveau/nv30/nv30_screen.c
+++ b/src/gallium/drivers/nouveau/nv30/nv30_screen.c
@@ -214,20 +214,21 @@ nv30_screen_get_param(struct pipe_screen *pscreen, enum 
pipe_cap param)
     case PIPE_CAP_TGSI_TEX_TXF_LZ:
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;

     case PIPE_CAP_VENDOR_ID:
        return 0x10de;
     case PIPE_CAP_DEVICE_ID: {
        uint64_t device_id;
        if (nouveau_getparam(dev, NOUVEAU_GETPARAM_PCI_DEVICE, &device_id)) {
           NOUVEAU_ERR("NOUVEAU_GETPARAM_PCI_DEVICE failed.\n");
           return -1;
        }
diff --git a/src/gallium/drivers/nouveau/nv50/nv50_screen.c 
b/src/gallium/drivers/nouveau/nv50/nv50_screen.c
index fbda514..8347a2c 100644
--- a/src/gallium/drivers/nouveau/nv50/nv50_screen.c
+++ b/src/gallium/drivers/nouveau/nv50/nv50_screen.c
@@ -266,20 +266,21 @@ nv50_screen_get_param(struct pipe_screen *pscreen, enum 
pipe_cap param)
     case PIPE_CAP_DOUBLES:
     case PIPE_CAP_INT64:
     case PIPE_CAP_INT64_DIVMOD:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;

     case PIPE_CAP_VENDOR_ID:
        return 0x10de;
     case PIPE_CAP_DEVICE_ID: {
        uint64_t device_id;
        if (nouveau_getparam(dev, NOUVEAU_GETPARAM_PCI_DEVICE, &device_id)) {
           NOUVEAU_ERR("NOUVEAU_GETPARAM_PCI_DEVICE failed.\n");
           return -1;
        }
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c 
b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
index 6a7a575..65d4d43 100644
--- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
+++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
@@ -295,20 +295,21 @@ nvc0_screen_get_param(struct pipe_screen *pscreen, enum 
pipe_cap param)
     case PIPE_CAP_PCI_DEVICE:
     case PIPE_CAP_PCI_FUNCTION:
     case PIPE_CAP_VIEWPORT_SUBPIXEL_BITS:
     case PIPE_CAP_TGSI_CAN_READ_OUTPUTS:
     case PIPE_CAP_NATIVE_FENCE_FD:
     case PIPE_CAP_GLSL_OPTIMIZE_CONSERVATIVELY:
     case PIPE_CAP_INT64_DIVMOD:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;

     case PIPE_CAP_VENDOR_ID:
        return 0x10de;
     case PIPE_CAP_DEVICE_ID: {
        uint64_t device_id;
        if (nouveau_getparam(dev, NOUVEAU_GETPARAM_PCI_DEVICE, &device_id)) {
           NOUVEAU_ERR("NOUVEAU_GETPARAM_PCI_DEVICE failed.\n");
           return -1;
        }
diff --git a/src/gallium/drivers/r300/r300_screen.c 
b/src/gallium/drivers/r300/r300_screen.c
index a994a05..4caa94f 100644
--- a/src/gallium/drivers/r300/r300_screen.c
+++ b/src/gallium/drivers/r300/r300_screen.c
@@ -236,20 +236,21 @@ static int r300_get_param(struct pipe_screen* pscreen, 
enum pipe_cap param)
          case PIPE_CAP_TGSI_TEX_TXF_LZ:
          case PIPE_CAP_TGSI_CLOCK:
          case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
          case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
          case PIPE_CAP_TGSI_BALLOT:
          case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
          case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
          case PIPE_CAP_POST_DEPTH_COVERAGE:
          case PIPE_CAP_BINDLESS_TEXTURE:
          case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+        case PIPE_CAP_NIR_LOWER_IO:
              return 0;

          /* SWTCL-only features. */
          case PIPE_CAP_PRIMITIVE_RESTART:
          case PIPE_CAP_USER_VERTEX_BUFFERS:
          case PIPE_CAP_TGSI_VS_WINDOW_SPACE_POSITION:
              return !r300screen->caps.has_tcl;

          /* HWTCL-only features / limitations. */
          case PIPE_CAP_VERTEX_BUFFER_OFFSET_4BYTE_ALIGNED_ONLY:
diff --git a/src/gallium/drivers/r600/r600_pipe.c 
b/src/gallium/drivers/r600/r600_pipe.c
index f09e468..86c7e92 100644
--- a/src/gallium/drivers/r600/r600_pipe.c
+++ b/src/gallium/drivers/r600/r600_pipe.c
@@ -392,20 +392,21 @@ static int r600_get_param(struct pipe_screen* pscreen, 
enum pipe_cap param)
         case PIPE_CAP_INT64_DIVMOD:
         case PIPE_CAP_TGSI_TEX_TXF_LZ:
         case PIPE_CAP_TGSI_CLOCK:
         case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
         case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
         case PIPE_CAP_TGSI_BALLOT:
         case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
         case PIPE_CAP_POST_DEPTH_COVERAGE:
         case PIPE_CAP_BINDLESS_TEXTURE:
         case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+       case PIPE_CAP_NIR_LOWER_IO:
                 return 0;

         case PIPE_CAP_DOUBLES:
                 if (rscreen->b.family == CHIP_ARUBA ||
                     rscreen->b.family == CHIP_CAYMAN ||
                     rscreen->b.family == CHIP_CYPRESS ||
                     rscreen->b.family == CHIP_HEMLOCK)
                         return 1;
                 return 0;

diff --git a/src/gallium/drivers/radeonsi/si_pipe.c 
b/src/gallium/drivers/radeonsi/si_pipe.c
index 05a4555..6727f84 100644
--- a/src/gallium/drivers/radeonsi/si_pipe.c
+++ b/src/gallium/drivers/radeonsi/si_pipe.c
@@ -585,20 +585,21 @@ static int si_get_param(struct pipe_screen* pscreen, enum 
pipe_cap param)
         case PIPE_CAP_TEXTURE_GATHER_OFFSETS:
         case PIPE_CAP_VERTEXID_NOBASE:
         case PIPE_CAP_PRIMITIVE_RESTART_FOR_PATCHES:
         case PIPE_CAP_MAX_WINDOW_RECTANGLES:
         case PIPE_CAP_NATIVE_FENCE_FD:
         case PIPE_CAP_TGSI_FS_FBFETCH:
         case PIPE_CAP_TGSI_MUL_ZERO_WINS:
         case PIPE_CAP_UMA:
         case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
         case PIPE_CAP_POST_DEPTH_COVERAGE:
+       case PIPE_CAP_NIR_LOWER_IO:
                 return 0;

         case PIPE_CAP_QUERY_BUFFER_OBJECT:
                 return si_have_tgsi_compute(sscreen);

         case PIPE_CAP_DRAW_PARAMETERS:
         case PIPE_CAP_MULTI_DRAW_INDIRECT:
         case PIPE_CAP_MULTI_DRAW_INDIRECT_PARAMS:
                 return sscreen->has_draw_indirect_multi;

diff --git a/src/gallium/drivers/softpipe/sp_screen.c 
b/src/gallium/drivers/softpipe/sp_screen.c
index 6ed2ce1..eadd476 100644
--- a/src/gallium/drivers/softpipe/sp_screen.c
+++ b/src/gallium/drivers/softpipe/sp_screen.c
@@ -300,20 +300,21 @@ softpipe_get_param(struct pipe_screen *screen, enum 
pipe_cap param)
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;
     case PIPE_CAP_SHADER_BUFFER_OFFSET_ALIGNMENT:
        return 4;
     }
     /* should only get here on unhandled cases */
     debug_printf("Unexpected PIPE_CAP %d query\n", param);
     return 0;
  }

  static int
diff --git a/src/gallium/drivers/svga/svga_screen.c 
b/src/gallium/drivers/svga/svga_screen.c
index 6126b55..01443a7 100644
--- a/src/gallium/drivers/svga/svga_screen.c
+++ b/src/gallium/drivers/svga/svga_screen.c
@@ -439,20 +439,21 @@ svga_get_param(struct pipe_screen *screen, enum pipe_cap 
param)
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;
     }

     debug_printf("Unexpected PIPE_CAP_ query %u\n", param);
     return 0;
  }


  static int
  vgpu9_get_shader_param(struct pipe_screen *screen,
diff --git a/src/gallium/drivers/swr/swr_screen.cpp 
b/src/gallium/drivers/swr/swr_screen.cpp
index b0796bb..1e51d88 100644
--- a/src/gallium/drivers/swr/swr_screen.cpp
+++ b/src/gallium/drivers/swr/swr_screen.cpp
@@ -335,20 +335,21 @@ swr_get_param(struct pipe_screen *screen, enum pipe_cap 
param)
     case PIPE_CAP_TGSI_CLOCK:
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;

     case PIPE_CAP_VENDOR_ID:
        return 0xFFFFFFFF;
     case PIPE_CAP_DEVICE_ID:
        return 0xFFFFFFFF;
     case PIPE_CAP_ACCELERATED:
        return 0;
     case PIPE_CAP_VIDEO_MEMORY: {
        /* XXX: Do we want to return the full amount of system memory ? */
diff --git a/src/gallium/drivers/vc4/vc4_screen.c 
b/src/gallium/drivers/vc4/vc4_screen.c
index 6acd9bf..f4a207f 100644
--- a/src/gallium/drivers/vc4/vc4_screen.c
+++ b/src/gallium/drivers/vc4/vc4_screen.c
@@ -122,20 +122,21 @@ vc4_screen_get_param(struct pipe_screen *pscreen, enum 
pipe_cap param)
          case PIPE_CAP_NPOT_TEXTURES:
          case PIPE_CAP_SHAREABLE_SHADERS:
          case PIPE_CAP_USER_CONSTANT_BUFFERS:
          case PIPE_CAP_TEXTURE_SHADOW_MAP:
          case PIPE_CAP_BLEND_EQUATION_SEPARATE:
          case PIPE_CAP_TWO_SIDED_STENCIL:
          case PIPE_CAP_TEXTURE_MULTISAMPLE:
          case PIPE_CAP_TEXTURE_SWIZZLE:
          case PIPE_CAP_GLSL_OPTIMIZE_CONSERVATIVELY:
          case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
+        case PIPE_CAP_NIR_LOWER_IO:
                  return 1;

                  /* lying for GL 2.0 */
          case PIPE_CAP_OCCLUSION_QUERY:
          case PIPE_CAP_POINT_SPRITE:
                  return 1;

          case PIPE_CAP_CONSTANT_BUFFER_OFFSET_ALIGNMENT:
                  return 256;

diff --git a/src/gallium/drivers/virgl/virgl_screen.c 
b/src/gallium/drivers/virgl/virgl_screen.c
index e155b4a..8a579a0 100644
--- a/src/gallium/drivers/virgl/virgl_screen.c
+++ b/src/gallium/drivers/virgl/virgl_screen.c
@@ -259,20 +259,21 @@ virgl_get_param(struct pipe_screen *screen, enum pipe_cap 
param)
     case PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE:
     case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE:
     case PIPE_CAP_TGSI_BALLOT:
     case PIPE_CAP_DOUBLES:
     case PIPE_CAP_TGSI_TES_LAYER_VIEWPORT:
     case PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX:
     case PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION:
     case PIPE_CAP_POST_DEPTH_COVERAGE:
     case PIPE_CAP_BINDLESS_TEXTURE:
     case PIPE_CAP_NIR_SAMPLERS_AS_DEREF:
+   case PIPE_CAP_NIR_LOWER_IO:
        return 0;
     case PIPE_CAP_VENDOR_ID:
        return 0x1af4;
     case PIPE_CAP_DEVICE_ID:
        return 0x1010;
     case PIPE_CAP_ACCELERATED:
        return 1;
     case PIPE_CAP_UMA:
     case PIPE_CAP_VIDEO_MEMORY:
        return 0;
diff --git a/src/gallium/include/pipe/p_defines.h 
b/src/gallium/include/pipe/p_defines.h
index 08690ba..460238e 100644
--- a/src/gallium/include/pipe/p_defines.h
+++ b/src/gallium/include/pipe/p_defines.h
@@ -774,20 +774,21 @@ enum pipe_cap
     PIPE_CAP_TGSI_CLOCK,
     PIPE_CAP_POLYGON_MODE_FILL_RECTANGLE,
     PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE,
     PIPE_CAP_TGSI_BALLOT,
     PIPE_CAP_TGSI_TES_LAYER_VIEWPORT,
     PIPE_CAP_CAN_BIND_CONST_BUFFER_AS_VERTEX,
     PIPE_CAP_ALLOW_MAPPED_BUFFERS_DURING_EXECUTION,
     PIPE_CAP_POST_DEPTH_COVERAGE,
     PIPE_CAP_BINDLESS_TEXTURE,
     PIPE_CAP_NIR_SAMPLERS_AS_DEREF,
+   PIPE_CAP_NIR_LOWER_IO,
  };

  #define PIPE_QUIRK_TEXTURE_BORDER_COLOR_SWIZZLE_NV50 (1 << 0)
  #define PIPE_QUIRK_TEXTURE_BORDER_COLOR_SWIZZLE_R600 (1 << 1)

  enum pipe_endian
  {
     PIPE_ENDIAN_LITTLE = 0,
     PIPE_ENDIAN_BIG = 1,
  #if defined(PIPE_ARCH_LITTLE_ENDIAN)
--
2.9.3

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 18/92] gallium: add PIPE_CAP_NIR_LOWER_IO

Reply via email to