[Mesa-dev] [PATCH] spirv: Add support for the HelperInvocation builtin

[Ian Romanick]

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

Did you submit a CTS bug?

On 08/21/2017 10:11 PM, Jason Ekstrand wrote:
> I have no idea how this got missed but it's been missing since forever.
> 
> Cc: mesa-stable at lists.freedesktop.org
> ---
>  src/compiler/spirv/vtn_variables.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c
> index 6a8776b..87cb935 100644
> --- a/src/compiler/spirv/vtn_variables.c
> +++ b/src/compiler/spirv/vtn_variables.c
> @@ -1121,6 +1121,10 @@ vtn_get_builtin_location(struct vtn_builder *b,
>        *location = FRAG_RESULT_DEPTH;
>        assert(*mode == nir_var_shader_out);
>        break;
> +   case SpvBuiltInHelperInvocation:
> +      *location = SYSTEM_VALUE_HELPER_INVOCATION;
> +      set_mode_system_value(mode);
> +      break;
>     case SpvBuiltInNumWorkgroups:
>        *location = SYSTEM_VALUE_NUM_WORK_GROUPS;
>        set_mode_system_value(mode);
> @@ -1177,7 +1181,6 @@ vtn_get_builtin_location(struct vtn_builder *b,
>        *location = SYSTEM_VALUE_VIEW_INDEX;
>        set_mode_system_value(mode);
>        break;
> -   case SpvBuiltInHelperInvocation:
>     default:
>        unreachable("unsupported builtin");
>     }
>