Commit d9c4871b authored by Kenneth Graunke's avatar Kenneth Graunke
Browse files

intel/fs: Fix opt_peephole_csel to not throw away saturates.

We were not copying the saturate bit from the original instruction
to the new replacement instruction.  This caused major misrendering
in DiRT Rally on iris, where comparisons leading to discards failed
due to the missing saturate, causing lots of extra garbage pixels to
be drawn in text rendering, trees, and so on.

This did not show up on i965 because st/nir performs a more aggressive
version of nir_opt_peephole_select, yielding more b32csel operations.

Fixes: 52c7df16 i965/fs: Merge CMP and SEL into CSEL on Gen8+
parent 775aabdd
Pipeline #23504 passed with stages
in 58 minutes and 53 seconds
......@@ -3133,6 +3133,7 @@ fs_visitor::opt_peephole_csel()
if (csel_inst != NULL) {
progress = true;
csel_inst->saturate = inst->saturate;
inst->remove(block);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment