Sounds good to me, go ahead and push the new version!
On Fri, May 6, 2016 at 10:44 AM, Juan A. Suarez Romero <jasua...@igalia.com> wrote: > On Fri, 2016-05-06 at 10:39 -0400, Lars Hamre wrote: >> 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 > > That's great! Precisely, I have here a version that fixes those two > test by doing exactly that :D > > > I can push a new version with that change, or if you prefer doing it > yourself, just let me know > > > J.A. > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
