[Mesa-dev] [PATCH] r600g: Implement ARB_derivative_control

[Marek Olšák]

Pushed, thanks.

Marek

On Fri, Aug 15, 2014 at 10:30 AM, Glenn Kennard <glenn.kennard at gmail.com> wrote:
> Requires Evergreen/Cayman
>
> Signed-off-by: Glenn Kennard <glenn.kennard at gmail.com>
> ---
> Changes since v1:
>  Move TGSI_OPCODE_DD[XY]_FINE in opcode tables to reflect
>  changed enum values.
>
> All ARB_derivative_control piglit tests pass on radeon 6670
>
>  docs/GL3.txt                           |  2 +-
>  src/gallium/drivers/r600/r600_pipe.c   |  2 +-
>  src/gallium/drivers/r600/r600_shader.c | 20 +++++++++++---------
>  3 files changed, 13 insertions(+), 11 deletions(-)
>
> diff --git a/docs/GL3.txt b/docs/GL3.txt
> index 12b8f62..b38e42c 100644
> --- a/docs/GL3.txt
> +++ b/docs/GL3.txt
> @@ -188,7 +188,7 @@ GL 4.5, GLSL 4.50:
>    GL_ARB_clip_control                                  not started
>    GL_ARB_conditional_render_inverted                   not started
>    GL_ARB_cull_distance                                 not started
> -  GL_ARB_derivative_control                            DONE (i965, nv50, nvc0)
> +  GL_ARB_derivative_control                            DONE (i965, nv50, nvc0, r600)
>    GL_ARB_direct_state_access                           not started
>    GL_ARB_get_texture_sub_image                         started (Brian Paul)
>    GL_ARB_shader_texture_image_samples                  not started
> diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r600_pipe.c
> index 8a5ba79..bf52a19 100644
> --- a/src/gallium/drivers/r600/r600_pipe.c
> +++ b/src/gallium/drivers/r600/r600_pipe.c
> @@ -305,6 +305,7 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
>         case PIPE_CAP_TGSI_VS_LAYER_VIEWPORT:
>         case PIPE_CAP_TEXTURE_GATHER_SM5:
>         case PIPE_CAP_TEXTURE_QUERY_LOD:
> +       case PIPE_CAP_TGSI_FS_FINE_DERIVATIVE:
>                 return family >= CHIP_CEDAR ? 1 : 0;
>         case PIPE_CAP_MAX_TEXTURE_GATHER_COMPONENTS:
>                 return family >= CHIP_CEDAR ? 4 : 0;
> @@ -319,7 +320,6 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
>         case PIPE_CAP_SAMPLE_SHADING:
>         case PIPE_CAP_TEXTURE_GATHER_OFFSETS:
>         case PIPE_CAP_DRAW_INDIRECT:
> -       case PIPE_CAP_TGSI_FS_FINE_DERIVATIVE:
>                 return 0;
>
>         /* Stream output. */
> diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c
> index 481e9eb..3f089b4 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -5319,6 +5319,11 @@ static int tgsi_tex(struct r600_shader_ctx *ctx)
>         tex.src_gpr = src_gpr;
>         tex.dst_gpr = ctx->file_offset[inst->Dst[0].Register.File] + inst->Dst[0].Register.Index;
>
> +       if (inst->Instruction.Opcode == TGSI_OPCODE_DDX_FINE ||
> +               inst->Instruction.Opcode == TGSI_OPCODE_DDY_FINE) {
> +               tex.inst_mod = 1; /* per pixel gradient calculation instead of per 2x2 quad */
> +       }
> +
>         if (inst->Instruction.Opcode == TGSI_OPCODE_TG4) {
>                 int8_t texture_component_select = ctx->literals[4 * inst->Src[1].Register.Index + inst->Src[1].Register.SwizzleX];
>                 tex.inst_mod = texture_component_select;
> @@ -6789,9 +6794,8 @@ static struct r600_shader_tgsi_instruction r600_shader_tgsi_instruction[] = {
>         {76,                    0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_ELSE,      0, ALU_OP0_NOP, tgsi_else},
>         {TGSI_OPCODE_ENDIF,     0, ALU_OP0_NOP, tgsi_endif},
> -       /* gap */
> -       {79,                    0, ALU_OP0_NOP, tgsi_unsupported},
> -       {80,                    0, ALU_OP0_NOP, tgsi_unsupported},
> +       {TGSI_OPCODE_DDX_FINE,  0, ALU_OP0_NOP, tgsi_unsupported},
> +       {TGSI_OPCODE_DDY_FINE,  0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_PUSHA,     0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_POPA,      0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_CEIL,      0, ALU_OP1_CEIL, tgsi_op2},
> @@ -6992,9 +6996,8 @@ static struct r600_shader_tgsi_instruction eg_shader_tgsi_instruction[] = {
>         {76,                    0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_ELSE,      0, ALU_OP0_NOP, tgsi_else},
>         {TGSI_OPCODE_ENDIF,     0, ALU_OP0_NOP, tgsi_endif},
> -       /* gap */
> -       {79,                    0, ALU_OP0_NOP, tgsi_unsupported},
> -       {80,                    0, ALU_OP0_NOP, tgsi_unsupported},
> +       {TGSI_OPCODE_DDX_FINE,  0, FETCH_OP_GET_GRADIENTS_H, tgsi_tex},
> +       {TGSI_OPCODE_DDY_FINE,  0, FETCH_OP_GET_GRADIENTS_V, tgsi_tex},
>         {TGSI_OPCODE_PUSHA,     0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_POPA,      0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_CEIL,      0, ALU_OP1_CEIL, tgsi_op2},
> @@ -7195,9 +7198,8 @@ static struct r600_shader_tgsi_instruction cm_shader_tgsi_instruction[] = {
>         {76,                    0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_ELSE,      0, ALU_OP0_NOP, tgsi_else},
>         {TGSI_OPCODE_ENDIF,     0, ALU_OP0_NOP, tgsi_endif},
> -       /* gap */
> -       {79,                    0, ALU_OP0_NOP, tgsi_unsupported},
> -       {80,                    0, ALU_OP0_NOP, tgsi_unsupported},
> +       {TGSI_OPCODE_DDX_FINE,  0, FETCH_OP_GET_GRADIENTS_H, tgsi_tex},
> +       {TGSI_OPCODE_DDY_FINE,  0, FETCH_OP_GET_GRADIENTS_V, tgsi_tex},
>         {TGSI_OPCODE_PUSHA,     0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_POPA,      0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_CEIL,      0, ALU_OP1_CEIL, tgsi_op2},
> --
> 1.8.3.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev