diff options
author | Iago Toral Quiroga <[email protected]> | 2018-03-07 11:12:18 +0100 |
---|---|---|
committer | Iago Toral Quiroga <[email protected]> | 2018-05-03 11:40:25 +0200 |
commit | 939501c8edc963c429f19eb43f798f6db343a65f (patch) | |
tree | 99a21f454554e9373b5bd66b721162db443d8d95 /src/intel | |
parent | d5a419176f20961babc78a6f1f319697c55e81ab (diff) |
intel/compiler: implement conversions from 16-bit int/float to bool
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/intel')
-rw-r--r-- | src/intel/compiler/brw_fs_nir.cpp | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp index 137e917f610..3b277959e95 100644 --- a/src/intel/compiler/brw_fs_nir.cpp +++ b/src/intel/compiler/brw_fs_nir.cpp @@ -1164,8 +1164,9 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr) break; case nir_op_i2b: - case nir_op_f2b: - if (nir_src_bit_size(instr->src[0].src) == 64) { + case nir_op_f2b: { + uint32_t bit_size = nir_src_bit_size(instr->src[0].src); + if (bit_size == 64) { /* two-argument instructions can't take 64-bit immediates */ fs_reg zero; fs_reg tmp; @@ -1187,13 +1188,18 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr) bld.CMP(tmp, op[0], zero, BRW_CONDITIONAL_NZ); bld.MOV(result, subscript(tmp, BRW_REGISTER_TYPE_UD, 0)); } else { - if (instr->op == nir_op_f2b) { - bld.CMP(result, op[0], brw_imm_f(0.0f), BRW_CONDITIONAL_NZ); + fs_reg zero; + if (bit_size == 32) { + zero = instr->op == nir_op_f2b ? brw_imm_f(0.0f) : brw_imm_d(0); } else { - bld.CMP(result, op[0], brw_imm_d(0), BRW_CONDITIONAL_NZ); + assert(bit_size == 16); + zero = instr->op == nir_op_f2b ? + retype(brw_imm_w(0), BRW_REGISTER_TYPE_HF) : brw_imm_w(0); } + bld.CMP(result, op[0], zero, BRW_CONDITIONAL_NZ); } break; + } case nir_op_ftrunc: inst = bld.RNDZ(result, op[0]); |