[Mesa-dev] [PATCH] r600g: Implement gpu_shader5 integer ops

Alex Deucher alexdeucher at gmail.com
Wed Jul 23 09:29:16 PDT 2014


On Wed, Jul 23, 2014 at 5:36 AM, Glenn Kennard <glenn.kennard at gmail.com> wrote:
> ---
> Together with separate MUL_HI/UMUL_HI patch this passes piglit
> ARB_gpu_shader5 integer tests.
>
> This patch trivially depends on r600g-Implement-GL_ARB_texture_query_lod
> for the TGSI_OPCODE_LODQ table entries.

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

>
>  docs/GL3.txt                           |   2 +-
>  src/gallium/drivers/r600/r600_shader.c | 190 +++++++++++++++++++++++++++++++++
>  2 files changed, 191 insertions(+), 1 deletion(-)
>
> diff --git a/docs/GL3.txt b/docs/GL3.txt
> index d481148..603413f 100644
> --- a/docs/GL3.txt
> +++ b/docs/GL3.txt
> @@ -105,7 +105,7 @@ GL 4.0:
>    - Dynamically uniform UBO array indices              started (Chris)
>    - Implicit signed -> unsigned conversions            DONE
>    - Fused multiply-add                                 DONE (i965, nvc0)
> -  - Packing/bitfield/conversion functions              DONE (i965, nvc0)
> +  - Packing/bitfield/conversion functions              DONE (i965, nvc0, r600)
>    - Enhanced textureGather                             DONE (i965, nvc0, radeonsi)
>    - Geometry shader instancing                         DONE (i965, nvc0)
>    - Geometry shader multiple streams                   DONE (i965, nvc0)
> diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c
> index 499e511..9abfee1 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -4192,6 +4192,172 @@ static int tgsi_ssg(struct r600_shader_ctx *ctx)
>         return 0;
>  }
>
> +static int tgsi_bfi(struct r600_shader_ctx *ctx)
> +{
> +       struct tgsi_full_instruction *inst = &ctx->parse.FullToken.FullInstruction;
> +       struct r600_bytecode_alu alu;
> +       int i, r, t1, t2;
> +
> +       unsigned write_mask = inst->Dst[0].Register.WriteMask;
> +       int last_inst = tgsi_last_instruction(write_mask);
> +
> +       t1 = ctx->temp_reg;
> +
> +       for (i = 0; i < 4; i++) {
> +               if (!(write_mask & (1<<i)))
> +                       continue;
> +
> +               /* create mask tmp */
> +               memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +               alu.op = ALU_OP2_BFM_INT;
> +               alu.dst.sel = t1;
> +               alu.dst.chan = i;
> +               alu.dst.write = 1;
> +               alu.last = i == last_inst;
> +
> +               r600_bytecode_src(&alu.src[0], &ctx->src[3], i);
> +               r600_bytecode_src(&alu.src[1], &ctx->src[2], i);
> +
> +               r = r600_bytecode_add_alu(ctx->bc, &alu);
> +               if (r)
> +                       return r;
> +       }
> +
> +       t2 = r600_get_temp(ctx);
> +
> +       for (i = 0; i < 4; i++) {
> +               if (!(write_mask & (1<<i)))
> +                       continue;
> +
> +               /* shift insert left */
> +               memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +               alu.op = ALU_OP2_LSHL_INT;
> +               alu.dst.sel = t2;
> +               alu.dst.chan = i;
> +               alu.dst.write = 1;
> +               alu.last = i == last_inst;
> +
> +               r600_bytecode_src(&alu.src[0], &ctx->src[1], i);
> +               r600_bytecode_src(&alu.src[1], &ctx->src[2], i);
> +
> +               r = r600_bytecode_add_alu(ctx->bc, &alu);
> +               if (r)
> +                       return r;
> +       }
> +
> +       for (i = 0; i < 4; i++) {
> +               if (!(write_mask & (1<<i)))
> +                       continue;
> +
> +               /* actual bitfield insert */
> +               memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +               alu.op = ALU_OP3_BFI_INT;
> +               alu.is_op3 = 1;
> +               tgsi_dst(ctx, &inst->Dst[0], i, &alu.dst);
> +               alu.dst.chan = i;
> +               alu.dst.write = 1;
> +               alu.last = i == last_inst;
> +
> +               alu.src[0].sel = t1;
> +               alu.src[0].chan = i;
> +               alu.src[1].sel = t2;
> +               alu.src[1].chan = i;
> +               r600_bytecode_src(&alu.src[2], &ctx->src[0], i);
> +
> +               r = r600_bytecode_add_alu(ctx->bc, &alu);
> +               if (r)
> +                       return r;
> +       }
> +
> +       return 0;
> +}
> +
> +static int tgsi_msb(struct r600_shader_ctx *ctx)
> +{
> +       struct tgsi_full_instruction *inst = &ctx->parse.FullToken.FullInstruction;
> +       struct r600_bytecode_alu alu;
> +       int i, r, t1, t2;
> +
> +       unsigned write_mask = inst->Dst[0].Register.WriteMask;
> +       int last_inst = tgsi_last_instruction(write_mask);
> +
> +       assert(ctx->inst_info->op == ALU_OP1_FFBH_INT ||
> +               ctx->inst_info->op == ALU_OP1_FFBH_UINT);
> +
> +       t1 = ctx->temp_reg;
> +
> +       /* bit position is indexed from lsb by TGSI, and from msb by the hardware */
> +       for (i = 0; i < 4; i++) {
> +               if (!(write_mask & (1<<i)))
> +                       continue;
> +
> +               /* t1 = FFBH_INT / FFBH_UINT */
> +               memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +               alu.op = ctx->inst_info->op;
> +               alu.dst.sel = t1;
> +               alu.dst.chan = i;
> +               alu.dst.write = 1;
> +               alu.last = i == last_inst;
> +
> +               r600_bytecode_src(&alu.src[0], &ctx->src[0], i);
> +
> +               r = r600_bytecode_add_alu(ctx->bc, &alu);
> +               if (r)
> +                       return r;
> +       }
> +
> +       t2 = r600_get_temp(ctx);
> +
> +       for (i = 0; i < 4; i++) {
> +               if (!(write_mask & (1<<i)))
> +                       continue;
> +
> +               /* t2 = 31 - t1 */
> +               memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +               alu.op = ALU_OP2_SUB_INT;
> +               alu.dst.sel = t2;
> +               alu.dst.chan = i;
> +               alu.dst.write = 1;
> +               alu.last = i == last_inst;
> +
> +               alu.src[0].sel = V_SQ_ALU_SRC_LITERAL;
> +               alu.src[0].value = 31;
> +               alu.src[1].sel = t1;
> +               alu.src[1].chan = i;
> +
> +               r = r600_bytecode_add_alu(ctx->bc, &alu);
> +               if (r)
> +                       return r;
> +       }
> +
> +       for (i = 0; i < 4; i++) {
> +               if (!(write_mask & (1<<i)))
> +                       continue;
> +
> +               /* result = t1 >= 0 ? t2 : t1 */
> +               memset(&alu, 0, sizeof(struct r600_bytecode_alu));
> +               alu.op = ALU_OP3_CNDGE_INT;
> +               alu.is_op3 = 1;
> +               tgsi_dst(ctx, &inst->Dst[0], i, &alu.dst);
> +               alu.dst.chan = i;
> +               alu.dst.write = 1;
> +               alu.last = i == last_inst;
> +
> +               alu.src[0].sel = t1;
> +               alu.src[0].chan = i;
> +               alu.src[1].sel = t2;
> +               alu.src[1].chan = i;
> +               alu.src[2].sel = t1;
> +               alu.src[2].chan = i;
> +
> +               r = r600_bytecode_add_alu(ctx->bc, &alu);
> +               if (r)
> +                       return r;
> +       }
> +
> +       return 0;
> +}
> +
>  static int tgsi_helper_copy(struct r600_shader_ctx *ctx, struct tgsi_full_instruction *inst)
>  {
>         struct r600_bytecode_alu alu;
> @@ -6678,6 +6844,14 @@ static struct r600_shader_tgsi_instruction r600_shader_tgsi_instruction[] = {
>         {TGSI_OPCODE_UMUL_HI, 0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_TG4,   0, FETCH_OP_GATHER4, tgsi_unsupported},
>         {TGSI_OPCODE_LODQ,      0, FETCH_OP_GET_LOD, tgsi_unsupported},
> +       {TGSI_OPCODE_IBFE,      1, ALU_OP3_BFE_INT, tgsi_unsupported},
> +       {TGSI_OPCODE_UBFE,      1, ALU_OP3_BFE_UINT, tgsi_unsupported},
> +       {TGSI_OPCODE_BFI,       0, ALU_OP0_NOP, tgsi_unsupported},
> +       {TGSI_OPCODE_BREV,      0, ALU_OP1_BFREV_INT, tgsi_unsupported},
> +       {TGSI_OPCODE_POPC,      0, ALU_OP1_BCNT_INT, tgsi_unsupported},
> +       {TGSI_OPCODE_LSB,       0, ALU_OP1_FFBL_INT, tgsi_unsupported},
> +       {TGSI_OPCODE_IMSB,      0, ALU_OP1_FFBH_INT, tgsi_unsupported},
> +       {TGSI_OPCODE_UMSB,      0, ALU_OP1_FFBH_UINT, tgsi_unsupported},
>         {TGSI_OPCODE_LAST,      0, ALU_OP0_NOP, tgsi_unsupported},
>  };
>
> @@ -6874,6 +7048,14 @@ static struct r600_shader_tgsi_instruction eg_shader_tgsi_instruction[] = {
>         {TGSI_OPCODE_UMUL_HI, 0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_TG4,   0, FETCH_OP_GATHER4, tgsi_tex},
>         {TGSI_OPCODE_LODQ,      0, FETCH_OP_GET_LOD, tgsi_tex},
> +       {TGSI_OPCODE_IBFE,      1, ALU_OP3_BFE_INT, tgsi_op3},
> +       {TGSI_OPCODE_UBFE,      1, ALU_OP3_BFE_UINT, tgsi_op3},
> +       {TGSI_OPCODE_BFI,       0, ALU_OP0_NOP, tgsi_bfi},
> +       {TGSI_OPCODE_BREV,      0, ALU_OP1_BFREV_INT, tgsi_op2},
> +       {TGSI_OPCODE_POPC,      0, ALU_OP1_BCNT_INT, tgsi_op2},
> +       {TGSI_OPCODE_LSB,       0, ALU_OP1_FFBL_INT, tgsi_op2},
> +       {TGSI_OPCODE_IMSB,      0, ALU_OP1_FFBH_INT, tgsi_msb},
> +       {TGSI_OPCODE_UMSB,      0, ALU_OP1_FFBH_UINT, tgsi_msb},
>         {TGSI_OPCODE_LAST,      0, ALU_OP0_NOP, tgsi_unsupported},
>  };
>
> @@ -7071,5 +7253,13 @@ static struct r600_shader_tgsi_instruction cm_shader_tgsi_instruction[] = {
>         {TGSI_OPCODE_UMUL_HI, 0, ALU_OP0_NOP, tgsi_unsupported},
>         {TGSI_OPCODE_TG4,   0, FETCH_OP_GATHER4, tgsi_tex},
>         {TGSI_OPCODE_LODQ,      0, FETCH_OP_GET_LOD, tgsi_tex},
> +       {TGSI_OPCODE_IBFE,      1, ALU_OP3_BFE_INT, tgsi_op3},
> +       {TGSI_OPCODE_UBFE,      1, ALU_OP3_BFE_UINT, tgsi_op3},
> +       {TGSI_OPCODE_BFI,       0, ALU_OP0_NOP, tgsi_bfi},
> +       {TGSI_OPCODE_BREV,      0, ALU_OP1_BFREV_INT, tgsi_op2},
> +       {TGSI_OPCODE_POPC,      0, ALU_OP1_BCNT_INT, tgsi_op2},
> +       {TGSI_OPCODE_LSB,       0, ALU_OP1_FFBL_INT, tgsi_op2},
> +       {TGSI_OPCODE_IMSB,      0, ALU_OP1_FFBH_INT, tgsi_msb},
> +       {TGSI_OPCODE_UMSB,      0, ALU_OP1_FFBH_UINT, tgsi_msb},
>         {TGSI_OPCODE_LAST,      0, ALU_OP0_NOP, tgsi_unsupported},
>  };
> --
> 1.8.3.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list