i965: Don't set EmitNoMainReturn.
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 5 Jan 2017 18:46:39 +0000 (10:46 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 6 Jan 2017 07:15:39 +0000 (23:15 -0800)
commita4d6f4d954d8539c3effbbc3af7f74e9dae32e2c
tree605a777a3484d09fd13e05803ba97804b3477e8d
parent69da8c32c78cf67a55f32c71932d89b01154d49c
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
tests/spec/glsl-1.10/execution/vs-nested-return-sibling-loop.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99287
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Timothy Arceri <timothy.arceri@collabora.com>
src/mesa/drivers/dri/i965/brw_compiler.c