summaryrefslogtreecommitdiffstats
path: root/src/util
diff options
context:
space:
mode:
authorKenneth Graunke <[email protected]>2016-07-26 13:19:46 -0700
committerKenneth Graunke <[email protected]>2016-07-29 16:05:24 -0700
commitebdc82d06532f992aea592265c29a11330e698fa (patch)
tree16e53a0aa2fc2d1abda460704dffc39faeb9e584 /src/util
parent591eeb7d1c98d85142453d6ba2b8128c9a5a554e (diff)
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 <[email protected]> Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/util')
0 files changed, 0 insertions, 0 deletions