Reviewed-by: Ian Romanick <ian.d.roman...@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-sta...@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");
> }
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
