On Thu, Oct 4, 2018 at 10:04 AM Matt Turner <matts...@gmail.com> wrote:

> Nice.
>
> Reviewed-by: Matt Turner <matts...@gmail.com>
>
> A piglit test would be a great addition as well.
>

As you wish:

https://patchwork.freedesktop.org/patch/254839/

That test doesn't actually trigger the bug though.  You have to hack the
driver up to do a round of copy-propagation at some point in the
compilation flow before the first alu_to_scalar.  If you do that, the patch
above will trigger the bug and passes with my patch.  Without such a driver
hack (or doing it in Vulkan we we do copy-prop before the main optimization
loop), it's nearly impossible to hit.

--Jason
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to