On Mon, 26 Sep 2011 16:51:34 -0700, Kenneth Graunke <kenn...@whitecape.org> wrote: > Signed-off-by: Kenneth Graunke <kenn...@whitecape.org> > > --- > src/mesa/drivers/dri/i965/brw_fs_emit.cpp | 14 ++++++++++---- > 1 files changed, 10 insertions(+), 4 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp > b/src/mesa/drivers/dri/i965/brw_fs_emit.cpp > index 8176a76..b35d558 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs_emit.cpp > @@ -151,7 +151,10 @@ fs_visitor::generate_math1_gen6(fs_inst *inst, > > assert(inst->mlen == 0); > > - brw_set_compression_control(p, BRW_COMPRESSION_NONE); > + bool force_simd8 = intel->gen < 7 || dst.type != BRW_REGISTER_TYPE_F; > + > + if (force_simd8) > + brw_set_compression_control(p, BRW_COMPRESSION_NONE);
I'm not following why force simd8 for dst.type != BRW_REGISTER_TYPE_F. Also, it looks like we could drop the math workaround stuff for the VS on IVB?
pgp7ztkLsPZw6.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
