[Mesa-dev] [PATCH 48/59] intel/compiler: implement isign for int8

[Jason Ekstrand]

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Tue, Dec 4, 2018 at 1:19 AM Iago Toral Quiroga <itoral at igalia.com> wrote:

> ---
>  src/intel/compiler/brw_fs_nir.cpp | 25 +++++++++++++++++++++----
>  1 file changed, 21 insertions(+), 4 deletions(-)
>
> diff --git a/src/intel/compiler/brw_fs_nir.cpp
> b/src/intel/compiler/brw_fs_nir.cpp
> index db3a8812ae3..7a4594a24ac 100644
> --- a/src/intel/compiler/brw_fs_nir.cpp
> +++ b/src/intel/compiler/brw_fs_nir.cpp
> @@ -1063,11 +1063,28 @@ fs_visitor::nir_emit_alu(const fs_builder &bld,
> nir_alu_instr *instr)
>         *  Predicated OR sets 1 if val is positive.
>         */
>        uint32_t bit_size = nir_dest_bit_size(instr->dest.dest);
> -      assert(bit_size == 32 || bit_size == 16);
>
> -      fs_reg zero = bit_size == 32 ? brw_imm_d(0) : brw_imm_w(0);
> -      fs_reg one = bit_size == 32 ? brw_imm_d(1) : brw_imm_w(1);
> -      fs_reg shift = bit_size == 32 ? brw_imm_d(31) : brw_imm_w(15);
> +      fs_reg zero, one, shift;
> +      switch (bit_size) {
> +      case 32:
> +         zero = brw_imm_d(0);
> +         one = brw_imm_d(1);
> +         shift = brw_imm_d(31);
> +         break;
> +      case 16:
> +         zero = brw_imm_w(0);
> +         one = brw_imm_w(1);
> +         shift = brw_imm_w(15);
> +         break;
> +      case 8: {
> +         zero = setup_imm_b(bld, 0);
> +         one = setup_imm_b(bld, 1);
> +         shift = setup_imm_b(bld, 7);
> +         break;
> +      }
> +      default:
> +         unreachable("unsupported bit-size");
> +      };
>
>        bld.CMP(bld.null_reg_d(), op[0], zero, BRW_CONDITIONAL_G);
>        bld.ASR(result, op[0], shift);
> --
> 2.17.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181207/1566d782/attachment.html>