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
On 06/10/17 14:07, Jason Ekstrand wrote:
Is there a test case for this?
Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net
<mailto:ja...@jlekstrand.net>>
On Fri, Oct 6, 2017 at 2:36 AM, Samuel Iglesias Gonsálvez
<sigles...@igalia.com <mailto:sigles...@igalia.com>> wrote:
SpvOpImageFetch doesn't provide it, so set it to zero.
Signed-off-by: Samuel Iglesias Gonsálvez <sigles...@igalia.com
<mailto:sigles...@igalia.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 <mailto:mesa-dev@lists.freedesktop.org>
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
<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
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev