fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-01-06 13:10:10 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

aco: set prefer_remove for gfx9- too

Browse source 
This is a hint that the branch is worth removing. Assume that's the case,
regardless of the gfx level.

fossil-db (vega10):
Totals from 22 (0.03% of 63053) affected shaders:
Instrs: 23927 -> 23856 (-0.30%)
CodeSize: 125096 -> 124812 (-0.23%)
Latency: 138258 -> 137765 (-0.36%)
InvThroughput: 55900 -> 55884 (-0.03%)
Branches: 391 -> 320 (-18.16%)

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/30321>
This commit is contained in:
Rhys Perry 2024-08-13 15:44:19 +01:00 • committed by Marge Bot
parent 9f1a5645cf
commit aafb49f56b
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/amd/compiler/aco_lower_to_hw_instr.cpp
View file

@ -2837,7 +2837,7 @@ lower_to_hw_instr(Program* program)
* - The application prefers to remove control flow
* - The compiler stack knows that it's a divergent branch always taken
*/
const bool prefer_remove = branch->rarely_taken && ctx.program->gfx_level >= GFX10;
const bool prefer_remove = branch->rarely_taken;
bool can_remove = block->index < target;
unsigned num_scalar = 0;
unsigned num_vector = 0;