On Sunday, August 10, 2014 04:34:36 PM Chris Forbes wrote: > > + if (align16) { > > + /* Emulate the Gen7 zeroing bug (see comments in vec4_visitor's > > + * emit_pack_half_2x16 method.) > > + */ > > + brw_MOV(p, retype(dst, BRW_REGISTER_TYPE_UD), brw_imm_ud(0u)); > > + } > > + return brw_MOV(p, retype(dst, BRW_REGISTER_TYPE_HF), src); > > + } else { > > + assert(brw->gen >= 7); > > This can be == 7. > > > + return brw_alu1(p, BRW_OPCODE_F32TO16, dst, src); > > + } > > +} > > + > > +brw_inst * > > +brw_F16TO32(struct brw_compile *p, struct brw_reg dst, struct brw_reg src) > > +{ > > + const struct brw_context *brw = p->brw; > > + assert(src.type == BRW_REGISTER_TYPE_W || > > + src.type == BRW_REGISTER_TYPE_UW || > > + src.type == BRW_REGISTER_TYPE_HF); > > + > > + if (brw->gen >= 8) { > > + return brw_MOV(p, dst, retype(src, BRW_REGISTER_TYPE_HF)); > > + } else { > > + assert(brw->gen >= 7); > > Same here
Good catch. Fixed in v2 - thanks. --Ken
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev