On Sat, Sep 20, 2014 at 10:23 AM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > Signed-off-by: Jason Ekstrand <jason.ekstr...@intel.com> > --- > src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 23 ++++++++++++++++++----- > 1 file changed, 18 insertions(+), 5 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > index f20a4a7..00cf3ec 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > @@ -1536,12 +1536,25 @@ fs_generator::generate_code(const cfg_t *cfg) > brw_set_default_mask_control(p, inst->force_writemask_all); > brw_set_default_acc_write_control(p, inst->writes_accumulator); > > - if (inst->force_uncompressed || dispatch_width == 8) { > + switch (inst->exec_size) { > + case 1: > + case 2: > + case 4: > + assert(inst->force_writemask_all); > brw_set_default_compression_control(p, BRW_COMPRESSION_NONE); > - } else if (inst->force_sechalf) { > - brw_set_default_compression_control(p, BRW_COMPRESSION_2NDHALF); > - } else { > - brw_set_default_compression_control(p, BRW_COMPRESSION_COMPRESSED); > + break; > + case 8: > + if (inst->force_sechalf) { > + brw_set_default_compression_control(p, BRW_COMPRESSION_2NDHALF); > + } else { > + brw_set_default_compression_control(p, BRW_COMPRESSION_NONE);
Indentation is off here, or in the block above. > + } > + break; > + case 16: > + brw_set_default_compression_control(p, BRW_COMPRESSION_COMPRESSED); > + break; > + default: > + assert(!"Invalid instruction width"); unreachable. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev