On Fri, 2017-10-06 at 14:23 +0100, Lionel Landwerlin wrote: > I fixed a similar bug in the vec4 > backend a couple of days ago. > > Can we maybe put this logic somewhere that could reused across > backends? > > Or maybe a nir pass to add the missing parameters? > > > > Thanks, > > > > - > > Lionel > >
Right. I think it should be reused across the backends, as it is the samplinginstruction the one who needs it; but I'm OK with either option. Sam > On 06/10/17 14:07, Jason Ekstrand wrote: > > > > > > > > Is there a test case for this? > > > > > > > > > > Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> > > > > > > > > > > On Fri, Oct 6, 2017 at 2:36 AM, Samuel > > Iglesias Gonsálvez <sigles...@igalia.com> > > wrote: > > > > > > > SpvOpImageFetch > > > doesn't provide it, so set it to zero. > > > > > > > > > > > > Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@i > > > galia.com> > > > > > > --- > > > > > > src/intel/compiler/brw_fs_nir.cpp | 9 +++++---- > > > > > > 1 file changed, 5 insertions(+), 4 deletions(-) > > > > > > > > > > > > diff --git a/src/intel/compiler/brw_fs_nir.cpp > > > b/src/intel/compiler/brw_fs_nir.cpp > > > > > > index 5b8ccd50bff..25488303c29 100644 > > > > > > --- a/src/intel/compiler/brw_fs_nir.cpp > > > > > > +++ b/src/intel/compiler/brw_fs_nir.cpp > > > > > > @@ -4518,11 +4518,12 @@ > > > fs_visitor::nir_emit_texture(const > > > fs_builder &bld, nir_tex_instr *instr) > > > > > > unreachable("unknown texture opcode"); > > > > > > } > > > > > > > > > > > > - /* TXS and TXL require a LOD but not everything > > > we > > > implement using those > > > > > > - * two opcodes provides one. Provide a default > > > LOD of > > > 0. > > > > > > + /* TXF, TXS and TXL require a LOD but not > > > everything we > > > implement using those > > > > > > + * three opcodes provides one. Provide a default > > > LOD of > > > 0. > > > > > > */ > > > > > > - if ((opcode == SHADER_OPCODE_TXS_LOGICAL || > > > > > > - opcode == SHADER_OPCODE_TXL_LOGICAL) && > > > > > > + if ((opcode == SHADER_OPCODE_TXF_LOGICAL || > > > > > > + opcode == SHADER_OPCODE_TXS_LOGICAL || > > > > > > + opcode == SHADER_OPCODE_TXL_LOGICAL) && > > > > > > srcs[TEX_LOGICAL_SRC_LOD].file == BAD_FILE) { > > > > > > srcs[TEX_LOGICAL_SRC_LOD] = brw_imm_ud(0u); > > > > > > } > > > > > > -- > > > > > > 2.13.6 > > > > > > > > > > > > _______________________________________________ > > > > > > mesa-dev mailing list > > > > > > mesa-dev@lists.freedesktop.org > > > > > > https://lists.freedesktop.org/mailman/listinfo/me > > > sa-dev > > > > > > > > > > > > > > > > > > > > > > > > > > > > _______________________________________________ > > mesa-dev mailing list > > mesa-dev@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > > > > > > > >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev