i965: Fix move_interpolation_to_top() pass.
authorKenneth Graunke <kenneth@whitecape.org>
Tue, 26 Jul 2016 20:19:46 +0000 (13:19 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Fri, 29 Jul 2016 23:05:24 +0000 (16:05 -0700)
commitebdc82d06532f992aea592265c29a11330e698fa
tree16e53a0aa2fc2d1abda460704dffc39faeb9e584
parent591eeb7d1c98d85142453d6ba2b8128c9a5a554e
i965: Fix move_interpolation_to_top() pass.

The pass I introduced in commit a2dc11a7818c04d8dc0324e8fcba98d60bae
was entirely broken.  A missing "break" made the load_interpolated_input
case always fall through to "default" and hit a "continue", making it
not actually move any load_interpolated_input intrinsics at all.
It would only move the simple load_barycentric_* intrinsics, which
don't emit any code anyway, making it basically useless.

The initial version I sent of the pass worked, but I apparently
failed to verify that the simplified version in v2 actually worked.

With the obvious fix applied (so we actually tried to move
load_interpolated_input intrinsics), I discovered a second bug: we
weren't moving the offset SSA def to the top, breaking SSA validation.

The new version of the pass actually moves load_interpolated_input
intrinsics and all their dependencies, as intended.

Papers over GPU hangs on Ivybridge and Baytrail caused by the
recent NIR FS input rework by restoring the old behavior.
(I'm not honestly sure why they hang with PLN not at the top.)

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97083
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/mesa/drivers/dri/i965/brw_fs.cpp