Media/intel-graphics-compiler
1
0
Fork 0
mirror of https://github.com/intel/intel-graphics-compiler.git synced 2025-11-04 08:21:06 +08:00
Code Issues Packages Projects Releases Wiki Activity

DeSSA uniform phi in divergent join 2nd

Browse Source 
should isolate uniform phi  instead of isolating other sources
This commit is contained in:
Gang Y Chen
2023-03-11 00:36:07 +00:00
committed by igcbot
parent 1123047dcf
commit 881acfa959
1 changed files with 2 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
IGC/Compiler/CISACodeGen/DeSSA.cpp
View File

@ -949,7 +949,6 @@ DeSSA::SplitInterferencesForBasicBlock(
// so we know MBB-target, i.e. the phi-block is a divergent-join
if (WIA->insideDivergentCF(MBB->getTerminator())) {
llvm::DenseMap<int, PHINode*> UniformColors;
for (BasicBlock::iterator BBI = (*SI)->begin(), BBE = (*SI)->end();
BBI != BBE; ++BBI) {
PHINode *PHI = dyn_cast<PHINode>(BBI);
@ -969,34 +968,8 @@ DeSSA::SplitInterferencesForBasicBlock(
Value *PredValue = PHI->getOperand(PredIndex);
// check if we will need to add a phi-copy from PBB
if (isa<Constant>(PredValue) || isIsolated(PredValue)) {
if (UniformColors.find(RootC) == UniformColors.end())
UniformColors[RootC] = PHI;
else
IGC_ASSERT(UniformColors[RootC] == PHI);
}
}
}
}
// After we find all the uniform-congruents that may have problem,
// Check if those colors are also used by the sources of other PHIs
// in this block, isolate those sources
for (BasicBlock::iterator BBI = (*SI)->begin(), BBE = (*SI)->end();
BBI != BBE; ++BBI) {
PHINode *PHI = dyn_cast<PHINode>(BBI);
if (!PHI)
isolateReg(PHI);
break;
unsigned PredIndex;
for (PredIndex = 0; PredIndex < PHI->getNumOperands();
++PredIndex) {
auto PBB = PHI->getIncomingBlock(PredIndex);
if (WIA->insideDivergentCF(PBB->getTerminator())) {
Value *PredValue = PHI->getOperand(PredIndex);
if (isa<Constant>(PredValue) || isIsolated(PredValue))
continue;
auto RootC = getRootColor(PredValue);
if (UniformColors.find(RootC) != UniformColors.end()) {
if (PHI != UniformColors[RootC])
isolateReg(PredValue);
}
}
}