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(0
> + 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_T
When we combine the Gen4-7 and Gen8+ generators, we'll need to handle
half float packing/unpacking functions somehow. The Gen8+ generator
code today just emulates the behavior of the Gen7 F32TO16/F16TO32
instructions, including the align16 mode bugs.
Rather than messing with fs_generator/vec4_gen