[Mesa-dev] [PATCH] spirv: Add support for the HelperInvocation builtin
Jason Ekstrand
jason at jlekstrand.net
Wed Aug 23 18:09:50 UTC 2017
On Wed, Aug 23, 2017 at 9:58 AM, Ian Romanick <idr at freedesktop.org> wrote:
> Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
>
> Did you submit a CTS bug?
>
No, I didn't. It does get some coverage through the up-and-coming subgroup
tests but it should probably have it's own test. That's going to be really
annoying to test...
> 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");
> > }
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170823/373543b2/attachment.html>
More information about the mesa-dev
mailing list