[Mesa-dev] [PATCH 1/2] intel/compiler: fix first_component for 64-bit types on vertex inputs

[Samuel Iglesias Gonsálvez]

Series is,

Reviewed-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>

Sam


On 19/01/18 09:17, Iago Toral Quiroga wrote:
> 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));
>        }

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180209/6a54419f/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180209/6a54419f/attachment.sig>