<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body text="#000000" bgcolor="#FFFFFF"> Series is, Reviewed-by: Samuel Iglesias Gonsálvez <a class="moz-txt-link-rfc2396E" href="mailto:siglesias@igalia.com"><siglesias@igalia.com></a> Sam <div class="moz-cite-prefix">On 19/01/18 09:17, Iago Toral Quiroga wrote: </div> <blockquote type="cite" cite="mid:20180119081707.5171-1-itoral@igalia.com"> <pre wrap="">Divide it by two as we do for other stages. This is because the component layout qualifier is always in 32-bit units. Fixes issues in a new CTS test (still WIP): KHR-GL45.enhanced_layouts.varying_double_components --- src/intel/compiler/brw_fs_nir.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp index 0d775649303..7a6346a4b5d 100644 --- a/src/intel/compiler/brw_fs_nir.cpp +++ b/src/intel/compiler/brw_fs_nir.cpp @@ -2420,6 +2420,9 @@ fs_visitor::nir_emit_vs_intrinsic(const fs_builder &bld, assert(const_offset && "Indirect input loads not allowed"); src = offset(src, bld, const_offset->u32[0]); + if (type_sz(type) == 8) + first_component /= 2; + for (unsigned j = 0; j < num_components; j++) { bld.MOV(offset(dest, bld, j), offset(src, bld, j + first_component)); } </pre> </blockquote> </body> </html>