AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

aco: don't CSE v_permlane across exec

Browse Source 
With bc=1 and fi=0 it needs to return 0 for inactive lanes.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26045>
This commit is contained in:
Georg Lehmann
2023-11-04 11:01:41 +01:00
committed by Marge Bot
parent a9bdf58c36
commit b33aa7b01a
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/amd/compiler/aco_opt_value_numbering.cpp
+5 -3
View File
@@ -166,15 +166,17 @@ struct InstrPred {
}
}
if (a->opcode == aco_opcode::v_readfirstlane_b32)
return a->pass_flags == b->pass_flags;
if (a->isVALU()) {
VALU_instruction& aV = a->valu();
VALU_instruction& bV = b->valu();
if (aV.abs != bV.abs || aV.neg != bV.neg || aV.clamp != bV.clamp || aV.omod != bV.omod ||
aV.opsel != bV.opsel || aV.opsel_lo != bV.opsel_lo || aV.opsel_hi != bV.opsel_hi)
return false;
if (a->opcode == aco_opcode::v_permlane16_b32 ||
a->opcode == aco_opcode::v_permlanex16_b32 ||
a->opcode == aco_opcode::v_readfirstlane_b32)
return aV.pass_flags == bV.pass_flags;
}
if (a->isDPP16()) {
DPP16_instruction& aDPP = a->dpp16();