Skip to content

Commit d718b8e

Browse files
bcheng0127igcbot
authored andcommitted
Fix for for send WAR dependence tracking
Fix for for send WAR dependence tracking
1 parent 04e2d4e commit d718b8e

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

visa/HWCaps.inc

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -853,17 +853,13 @@ bool noSrc0Src1OverlapSend() const {
853853

854854

855855
bool WARLocalization() const {
856-
if (getOption(vISA_SWSBMakeLocalWAR) && (getPlatform() >= Xe2)) {
856+
if (getOption(vISA_SWSBMakeLocalWAR)) {
857857
return true;
858858
}
859859

860860
return false;
861861
}
862862

863-
bool hasPVCSendWARWA() const {
864-
return hasDFInst() && (getPlatform() == Xe_PVC || getPlatform() == Xe_PVCXT);
865-
}
866-
867863
bool PVCSendWARWA() const {
868864
if (getOption(vISA_PVCSendWARWA) && (getPlatform() == Xe_PVC || getPlatform() == Xe_PVCXT)) {
869865
return true;

visa/LocalScheduler/SWSB_G4IR.cpp

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4902,19 +4902,13 @@ void SWSB::insertPVCWA() {
49024902

49034903
auto cleanLSCGRFs = [&](uint32_t &WATokens) {
49044904
for (uint32_t i = 0; i < totalTokenNum; i++) {
4905-
if (LSCLastTwoGRFsOfToken[i].first != INVALID_GRF) {
4906-
WATokens |= 1 << i;
4907-
}
49084905
LSCLastTwoGRFsOfToken[i].first = INVALID_GRF;
49094906
LSCLastTwoGRFsOfToken[i].second = INVALID_GRF;
49104907
}
49114908
};
49124909

49134910
auto cleanNonLSCGRFs = [&](uint32_t &WATokens) {
49144911
for (uint32_t i = 0; i < totalTokenNum; i++) {
4915-
if (nonLSClastTwoGRFsOfToken[i].first != INVALID_GRF) {
4916-
WATokens |= 1 << i;
4917-
}
49184912
nonLSClastTwoGRFsOfToken[i].first = INVALID_GRF;
49194913
nonLSClastTwoGRFsOfToken[i].second = INVALID_GRF;
49204914
}
@@ -4999,6 +4993,7 @@ void SWSB::insertPVCWA() {
49994993
if (LSCLastToken != UNKNOWN_TOKEN) {
50004994
if (insertDummyMovs(bb, inst_it, LSCLastToken, LSCLastTwoGRFsOfToken,
50014995
nonLSClastTwoGRFsOfToken)) {
4996+
WAtokens |= 1 << LSCLastToken;
50024997
cleanLSCGRFs(WAtokens);
50034998
}
50044999
}
@@ -5008,6 +5003,7 @@ void SWSB::insertPVCWA() {
50085003
if (insertDummyMovs(bb, inst_it, nonLSCLastToken,
50095004
LSCLastTwoGRFsOfToken,
50105005
nonLSClastTwoGRFsOfToken)) {
5006+
WAtokens |= 1 << nonLSCLastToken;
50115007
cleanNonLSCGRFs(WAtokens);
50125008
}
50135009
}

0 commit comments

Comments
 (0)