Hi Juan,
Sorry I missed that.
It looks like your patch doesn't fix the out parameter indexing for:
vs-inout-index-inout-mat2-col
vs-inout-index-inout-vec4-array
I was able to extend your patch to get these tests passing by:
- if the ir_array->array was a dereferenced_array, copy propagate it's index
- if the ir->ir_type == ir_type_swizzle, if it's val was a dereferenced_array,
copying propagate it's index
I would rather move ahead with your patch as mine seems quite
"hacky" in comparison :)
Regards,
Lars Hamre
On Fri, May 6, 2016 at 9:53 AM, Juan A. Suarez Romero
<jasua...@igalia.com> wrote:
> On Fri, 2016-05-06 at 08:49 -0400, Lars Hamre wrote:
>> Inout parameters which depended on other inout parameters
>> where not assigned in the correct order.
>>
>> Fixes the following piglit tests in shaders/out-parameter-indexing:
>> vs-inout-index-inout-float-array
>> vs-inout-index-inout-mat2-col
>> vs-inout-index-inout-mat2-row
>> vs-inout-index-inout-vec4
>> vs-inout-index-inout-vec4-array
>> vs-inout-index-inout-vec4-array-element
>
>
> Some days ago I sent a patch that also fixes some of these tests in a
> different way.
>
> https://lists.freedesktop.org/archives/mesa-dev/2016-May/115621.html
>
>
>
> J.A.
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
