The series looks good to me, with the comment on patch 5 fixed. Reviewed-by: Bas Nieuwenhuizen <[email protected]>
for the series. However, I'd be a lot more comfortable landing this if it sees a CTS run. Will kick one off tonight and see what happens. On Sun, Oct 15, 2017 at 1:36 AM, Timothy Arceri <[email protected]> wrote: > This initial linking support removes unused varyings across > stages. > > Future improvements include enabling removing unused varying > components [1], varying packing (in progress), and varying > array splitting. > > I've tried to run the series on the public Vulkan CTS but I > seem to be hitting an existing issue that locks up my machine [2]. > > Sascha Willems demo results: > > computecullandlod 39 -> 41 fps > pipelines ~6100 -> ~6200 fps > > 5 changed files with 375 additions and 406 deletions. > > [1] https://patchwork.freedesktop.org/series/31781/ > [2] > dEQP-VK.pipeline.render_to_image.core.1d_array.huge.width_layers.r8g8b8a8_unorm_d32_sfloat_s8_uint > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
