jay: swap predication/acc pass order

Lets us use more accumulators, I think this is well motivated. Saw this in a
test shader.

Totals from 242 (9.14% of 2647) affected shaders:
Instrs: 1365060 -> 1365035 (-0.00%); split: -0.00%, +0.00%
CodeSize: 20678592 -> 20680096 (+0.01%); split: -0.01%, +0.02%

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/41510>
This commit is contained in:
Alyssa Rosenzweig 2026-05-07 10:12:47 -04:00 committed by Marge Bot
parent afa9ab41b9
commit 9dbaaecb74

View file

@ -2677,8 +2677,12 @@ jay_compile(const struct intel_device_info *devinfo,
nir->info.bit_sizes_float);
if (!(jay_debug & JAY_DBG_NOOPT)) {
JAY_PASS(s, jay_opt_predicate);
/* jay_assign_accumulators uses a conservative liveness analysis for
* predication, so assign accumulators before predicating for better
* results.
*/
JAY_PASS(s, jay_assign_accumulators);
JAY_PASS(s, jay_opt_predicate);
}
JAY_PASS(s, jay_lower_scoreboard);