This test, along with piglit.spec.glsl-1_10.execution.vs-nested-return-sibling-loop2, regressed on the series ending with:
Author: Marek Olšák <firstname.lastname@example.org>
st/mesa: enable GLSLOptimizeConservatively for drivers that want it
Platforms: SNB and later
/tmp/build_root/m64/lib/piglit/bin/shader_runner /tmp/build_root/m64/lib/piglit/tests/spec/glsl-1.10/execution/vs-nested-return-sibling-loop.shader_test -auto
Probe color at (0,0)
Expected: 255 255 0 255
Observed: 0 255 0 255
Test failure on line 46
This is the only commit from the series that might affect i965:
Author: Marek Olšák <email@example.com>
Date: Sat Dec 31 12:02:26 2016 +0100
glsl: run do_lower_jumps properly in do_common_optimizations
I have a patch to fix this. Waiting on test results before sending.
Trivial patch on list:
Author: Kenneth Graunke <firstname.lastname@example.org>
Date: Thu Jan 5 10:46:39 2017 -0800
i965: Don't set EmitNoMainReturn.
A while ago, we stopped using Luca's GLSL IR lower_jumps pass in favor
of nir_lower_returns(). Marek's commit d3cb79e043338b0e55a3fba8df652f3
put it in do_common_optimization, which resulted in us calling it again.
Dropping the EmitNoMainReturn setting makes us skip that pass again.
Apparently that pass doesn't work properly, because this fixes Piglit's
Signed-off-by: Kenneth Graunke <email@example.com>
Reviewed-by: Timothy Arceri <firstname.lastname@example.org>