<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jan 9, 2017 at 6:57 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 10 January 2017 at 02:05, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Also, we should probably consider converting it over to a system value one<br>
> of these days.  That would make more sense from a theoretical model<br>
> perspective.  Not sure if it makes sense in any drivers though.<br>
><br>
> On Mon, Jan 9, 2017 at 8:03 AM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
>><br>
>> On Mon, Jan 9, 2017 at 4:50 AM, Iago Toral Quiroga <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>><br>
>> wrote:<br>
>>><br>
>>> Geometry and Tessellation stages do handle this as a system value<br>
>>> instead.<br>
>>><br>
>>> Fixes:<br>
>>> dEQP-VK.geometry.basic.<wbr>primitive_id<br>
>>> ---<br>
>>>  src/compiler/spirv/vtn_<wbr>variables.c | 3 ++-<br>
>>>  1 file changed, 2 insertions(+), 1 deletion(-)<br>
>>><br>
>>> diff --git a/src/compiler/spirv/vtn_<wbr>variables.c<br>
>>> b/src/compiler/spirv/vtn_<wbr>variables.c<br>
>>> index 9b84c97..cf47e5d 100644<br>
>>> --- a/src/compiler/spirv/vtn_<wbr>variables.c<br>
>>> +++ b/src/compiler/spirv/vtn_<wbr>variables.c<br>
>>> @@ -908,7 +908,8 @@ vtn_get_builtin_location(<wbr>struct vtn_builder *b,<br>
>>>        set_mode_system_value(mode);<br>
>>>        break;<br>
>>>     case SpvBuiltInPrimitiveId:<br>
>>> -      if (*mode == nir_var_shader_out) {<br>
>>> +      if (*mode == nir_var_shader_out ||<br>
>>> +          b->shader->stage == MESA_SHADER_FRAGMENT) {<br>
>><br>
>><br>
>> Can we do this instead:<br>
<br>
</div></div>I've had this in my geom shader tree exactly the same patch, I thought I'd<br>
sent it,<br>
<br>
but I think Jason code is cleaner so maybe use that,<br>
<br>
Reviewed-by: Dave Airlie <<a href="mailto:ailried@redhat.com">ailried@redhat.com</a>><br>
<br>
However I don't think this makes any sense as a system value,<br>
at least on amd hw it comes as an input.<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>I think we have it as an input too. :-)   I was mostly saying that because it seemed like a system-value sort of thing not because it would be more convenient.  If it's an input for both of us, we might as well leave it that way.<br></div></div></div></div>