On Thu, Sep 10, 2015 at 3:27 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > Please add the shader-db reults to the commit message. With that, > > Reviewed-by: Jason Ekstrand <jason.ekstr...@intel.com> > > Also, please CC stable on this one as well. I think there's some bugs > without it.
In particular it fixes https://bugs.freedesktop.org/show_bug.cgi?id=91716 so please add the following tag as well: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91716 before pushing --Jason > --Jason > > On Mon, Aug 24, 2015 at 4:51 AM, Antia Puentes <apuen...@igalia.com> wrote: >> Loads constants using integer as their register type, this is done >> for consistency with the FS backend. >> --- >> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp >> b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp >> index 632e409..23b2fab 100644 >> --- a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp >> +++ b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp >> @@ -456,7 +456,7 @@ void >> vec4_visitor::nir_emit_load_const(nir_load_const_instr *instr) >> { >> dst_reg reg = dst_reg(GRF, alloc.allocate(1)); >> - reg.type = BRW_REGISTER_TYPE_F; >> + reg.type = BRW_REGISTER_TYPE_D; >> >> unsigned remaining = brw_writemask_for_size(instr->def.num_components); >> >> @@ -477,7 +477,7 @@ vec4_visitor::nir_emit_load_const(nir_load_const_instr >> *instr) >> } >> >> reg.writemask = writemask; >> - emit(MOV(reg, src_reg(instr->value.f[i]))); >> + emit(MOV(reg, src_reg(instr->value.i[i]))); >> >> remaining &= ~writemask; >> } >> -- >> 2.1.0 >> >> _______________________________________________ >> mesa-dev mailing list >> mesa-dev@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
