Skip to content

Commit 03d0eac

Browse files
jgu222igcbot
authored andcommitted
Minor refactor
Change function name "insts" to "inst" to reflect what it means.
1 parent 4dd81d4 commit 03d0eac

File tree

3 files changed

+13
-13
lines changed

3 files changed

+13
-13
lines changed

visa/GraphColor.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6917,7 +6917,7 @@ unsigned GlobalRA::getRegionDisp(
69176917
return rowOffset + columnOffset;
69186918
}
69196919

6920-
void GlobalRA::addEUFusionCallWAInsts(G4_INST* inst)
6920+
void GlobalRA::addEUFusionCallWAInst(G4_INST* inst)
69216921
{
69226922
if(EUFusionCallWANeeded())
69236923
EUFusionCallWAInsts.insert(inst);
@@ -7677,7 +7677,7 @@ void GlobalRA::stackCallProlog()
76777677
entryBB->remove(oldSaveInst);
76787678
}
76797679

7680-
addEUFusionCallWAInsts(store);
7680+
addEUFusionCallWAInst(store);
76817681

76827682
return;
76837683
}
@@ -8371,8 +8371,8 @@ void GlobalRA::addCalleeStackSetupCode()
83718371
builder.kernel.getKernelDebugInfo()->setFrameSize(frameSize * 16);
83728372
}
83738373

8374-
addEUFusionCallWAInsts(createBEFP);
8375-
addEUFusionCallWAInsts(addInst);
8374+
addEUFusionCallWAInst(createBEFP);
8375+
addEUFusionCallWAInst(addInst);
83768376

83778377
if (EUFusionCallWANeeded())
83788378
{
@@ -8719,7 +8719,7 @@ void GlobalRA::addStoreRestoreToReturn()
87198719
load = builder.createFill(dstData, G4_ExecSize(execSize), 1, 0, builder.getBEFP(), InstOpt_WriteEnable, false);
87208720
}
87218721
fretBB->insertBefore(iter, load);
8722-
addEUFusionCallWAInsts(load);
8722+
addEUFusionCallWAInst(load);
87238723
restoreBE_FPInst = load;
87248724
}
87258725

visa/GraphColor.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -694,7 +694,7 @@ namespace vISA
694694
bool m_EUFusionCallWANeeded;
695695
public:
696696
bool EUFusionCallWANeeded() const { return m_EUFusionCallWANeeded; }
697-
void addEUFusionCallWAInsts(G4_INST* inst);
697+
void addEUFusionCallWAInst(G4_INST* inst);
698698
void removeEUFusionCallWAInst(G4_INST* inst) { EUFusionCallWAInsts.erase(inst); }
699699
const std::unordered_set<G4_INST*>& getEUFusionCallWAInsts() { return EUFusionCallWAInsts; }
700700
public:

visa/SpillManagerGMRF.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4918,7 +4918,7 @@ void GlobalRA::saveRestoreA0(G4_BB * bb)
49184918
bb->insertBefore(instIt, a0SSO);
49194919
if (EUFusionCallWANeeded())
49204920
{
4921-
addEUFusionCallWAInsts(a0SSO);
4921+
addEUFusionCallWAInst(a0SSO);
49224922
}
49234923
hasActiveSpillFill = true;
49244924
}
@@ -5143,7 +5143,7 @@ void GlobalRA::expandSpillLSC(G4_BB* bb, INST_LIST_ITER& instIt)
51435143
{
51445144
removeEUFusionCallWAInst(inst);
51455145
for (auto inst : builder->instList)
5146-
addEUFusionCallWAInsts(inst);
5146+
addEUFusionCallWAInst(inst);
51475147
}
51485148

51495149
splice(bb, instIt, builder->instList, inst->getCISAOff());
@@ -5226,7 +5226,7 @@ void GlobalRA::expandFillLSC(G4_BB* bb, INST_LIST_ITER& instIt)
52265226
{
52275227
removeEUFusionCallWAInst(inst);
52285228
for (auto inst : builder->instList)
5229-
addEUFusionCallWAInsts(inst);
5229+
addEUFusionCallWAInst(inst);
52305230
}
52315231

52325232
splice(bb, instIt, builder->instList, inst->getCISAOff());
@@ -5404,9 +5404,9 @@ void GlobalRA::expandSpillStackcall(
54045404
if (getEUFusionCallWAInsts().count(inst) > 0)
54055405
{
54065406
removeEUFusionCallWAInst(inst);
5407-
addEUFusionCallWAInsts(spillSends);
5407+
addEUFusionCallWAInst(spillSends);
54085408
if (hdrSetInst)
5409-
addEUFusionCallWAInsts(hdrSetInst);
5409+
addEUFusionCallWAInst(hdrSetInst);
54105410
}
54115411

54125412
if (kernel.getOption(vISA_GenerateDebugInfo))
@@ -5637,9 +5637,9 @@ void GlobalRA::expandFillStackcall(uint32_t numRows, uint32_t offset, short rowO
56375637
if (getEUFusionCallWAInsts().count(inst) > 0)
56385638
{
56395639
removeEUFusionCallWAInst(inst);
5640-
addEUFusionCallWAInsts(fillSends);
5640+
addEUFusionCallWAInst(fillSends);
56415641
if (hdrSetInst)
5642-
addEUFusionCallWAInsts(hdrSetInst);
5642+
addEUFusionCallWAInst(hdrSetInst);
56435643
}
56445644

56455645
std::stringstream comments;

0 commit comments

Comments
 (0)