[Mesa-dev] [PATCH 1/7] spirv/nir: update Xfb decoration comment

Jason Ekstrand jason at jlekstrand.net
Thu Nov 8 21:39:44 UTC 2018


On Thu, Nov 8, 2018 at 7:22 AM Alejandro PiƱeiro <apinheiro at igalia.com>
wrote:

> Although it is true that Vulkan doesn't support transform feedback
> yet, spirv to nir is handling it due ARB_gl_spirv support. Having said
> so, those decorations are handled elsewhere.
>

Actually, the RADV guys shipped their patches so.... This is especially
bogus. :)


> ---
>  src/compiler/spirv/spirv_to_nir.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/compiler/spirv/spirv_to_nir.c
> b/src/compiler/spirv/spirv_to_nir.c
> index 96ff09c3659..140e98eba27 100644
> --- a/src/compiler/spirv/spirv_to_nir.c
> +++ b/src/compiler/spirv/spirv_to_nir.c
> @@ -774,7 +774,7 @@ struct_member_decoration_cb(struct vtn_builder *b,
>
>     case SpvDecorationXfbBuffer:
>     case SpvDecorationXfbStride:
> -      vtn_warn("Vulkan does not have transform feedback");
> +      /* Handled at vtn_variables.c, apply_var_decoration */
>        break;
>
>     case SpvDecorationCPacked:
> --
> 2.14.1
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181108/818a3137/attachment.html>


More information about the mesa-dev mailing list