OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

i965/vec4: Get rid of the type parameter from to/from_double

Browse Source 
Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
This commit is contained in:
Jason Ekstrand
2017-03-07 18:32:17 -08:00
parent 702d1af8ba
commit bab4610e9c
2 changed files with 15 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/intel/compiler/brw_vec4.h
View File

@@ -321,10 +321,8 @@ public:
bool optimize_predicate(nir_alu_instr *instr, enum brw_predicate *predicate); bool optimize_predicate(nir_alu_instr *instr, enum brw_predicate *predicate);
void emit_conversion_from_double(dst_reg dst, src_reg src, bool saturate, void emit_conversion_from_double(dst_reg dst, src_reg src, bool saturate);
brw_reg_type single_type); void emit_conversion_to_double(dst_reg dst, src_reg src, bool saturate);
void emit_conversion_to_double(dst_reg dst, src_reg src, bool saturate,
brw_reg_type single_type);
src_reg setup_imm_df(double v); src_reg setup_imm_df(double v);

33
src/intel/compiler/brw_vec4_nir.cpp
View File

@@ -1170,14 +1170,13 @@ emit_find_msb_using_lzd(const vec4_builder &bld,
void void
vec4_visitor::emit_conversion_from_double(dst_reg dst, src_reg src, vec4_visitor::emit_conversion_from_double(dst_reg dst, src_reg src,
bool saturate, bool saturate)
brw_reg_type single_type)
{ {
/* BDW PRM vol 15 - workarounds: /* BDW PRM vol 15 - workarounds:
* DF->f format conversion for Align16 has wrong emask calculation when * DF->f format conversion for Align16 has wrong emask calculation when
* source is immediate. * source is immediate.
*/ */
if (devinfo->gen == 8 && single_type == BRW_REGISTER_TYPE_F && if (devinfo->gen == 8 && dst.type == BRW_REGISTER_TYPE_F &&
src.file == BRW_IMMEDIATE_VALUE) { src.file == BRW_IMMEDIATE_VALUE) {
vec4_instruction *inst = emit(MOV(dst, brw_imm_f(src.df))); vec4_instruction *inst = emit(MOV(dst, brw_imm_f(src.df)));
inst->saturate = saturate; inst->saturate = saturate;
@@ -1188,7 +1187,7 @@ vec4_visitor::emit_conversion_from_double(dst_reg dst, src_reg src,
emit(MOV(temp, src)); emit(MOV(temp, src));
dst_reg temp2 = dst_reg(this, glsl_type::dvec4_type); dst_reg temp2 = dst_reg(this, glsl_type::dvec4_type);
temp2 = retype(temp2, single_type); temp2 = retype(temp2, dst.type);
emit(VEC4_OPCODE_FROM_DOUBLE, temp2, src_reg(temp)) emit(VEC4_OPCODE_FROM_DOUBLE, temp2, src_reg(temp))
->size_written = 2 * REG_SIZE; ->size_written = 2 * REG_SIZE;
@@ -1198,12 +1197,11 @@ vec4_visitor::emit_conversion_from_double(dst_reg dst, src_reg src,
void void
vec4_visitor::emit_conversion_to_double(dst_reg dst, src_reg src, vec4_visitor::emit_conversion_to_double(dst_reg dst, src_reg src,
bool saturate, bool saturate)
brw_reg_type single_type)
{ {
dst_reg tmp_dst = dst_reg(src_reg(this, glsl_type::dvec4_type)); dst_reg tmp_dst = dst_reg(src_reg(this, glsl_type::dvec4_type));
src_reg tmp_src = retype(src_reg(this, glsl_type::vec4_type), single_type); src_reg tmp_src = retype(src_reg(this, glsl_type::vec4_type), src.type);
emit(MOV(dst_reg(tmp_src), retype(src, single_type))); emit(MOV(dst_reg(tmp_src), src));
emit(VEC4_OPCODE_TO_DOUBLE, tmp_dst, tmp_src); emit(VEC4_OPCODE_TO_DOUBLE, tmp_dst, tmp_src);
vec4_instruction *inst = emit(MOV(dst, src_reg(tmp_dst))); vec4_instruction *inst = emit(MOV(dst, src_reg(tmp_dst)));
inst->saturate = saturate; inst->saturate = saturate;
@@ -1301,27 +1299,21 @@ vec4_visitor::nir_emit_alu(nir_alu_instr *instr)
break; break;
case nir_op_d2f: case nir_op_d2f:
emit_conversion_from_double(dst, op[0], instr->dest.saturate, emit_conversion_from_double(dst, op[0], instr->dest.saturate);
BRW_REGISTER_TYPE_F);
break; break;
case nir_op_f2d: case nir_op_f2d:
emit_conversion_to_double(dst, op[0], instr->dest.saturate, emit_conversion_to_double(dst, op[0], instr->dest.saturate);
BRW_REGISTER_TYPE_F);
break; break;
case nir_op_d2i: case nir_op_d2i:
case nir_op_d2u: case nir_op_d2u:
emit_conversion_from_double(dst, op[0], instr->dest.saturate, emit_conversion_from_double(dst, op[0], instr->dest.saturate);
instr->op == nir_op_d2i ? BRW_REGISTER_TYPE_D :
BRW_REGISTER_TYPE_UD);
break; break;
case nir_op_i2d: case nir_op_i2d:
case nir_op_u2d: case nir_op_u2d:
emit_conversion_to_double(dst, op[0], instr->dest.saturate, emit_conversion_to_double(dst, op[0], instr->dest.saturate);
instr->op == nir_op_i2d ? BRW_REGISTER_TYPE_D :
BRW_REGISTER_TYPE_UD);
break; break;
case nir_op_iadd: case nir_op_iadd:
@@ -1968,8 +1960,9 @@ vec4_visitor::nir_emit_alu(nir_alu_instr *instr)
inst->predicate = BRW_PREDICATE_NORMAL; inst->predicate = BRW_PREDICATE_NORMAL;
/* Now convert the result from float to double */ /* Now convert the result from float to double */
emit_conversion_to_double(dst, src_reg(tmp), instr->dest.saturate, emit_conversion_to_double(dst, retype(src_reg(tmp),
BRW_REGISTER_TYPE_F); BRW_REGISTER_TYPE_F),
instr->dest.saturate);
} }
break; break;