Skip to content

Commit 4fc519a

Browse files
bcheng0127igcbot
authored andcommitted
BCR revert 204888
BCR revert 204888
1 parent 964f83b commit 4fc519a

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

visa/GraphColor.cpp

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -846,12 +846,10 @@ void BankConflictPass::setupBankConflictsforMad(G4_INST *inst) {
846846
G4_Declare *dcls[3];
847847
G4_Declare *opndDcls[3];
848848
BankConflict assignedBank = BANK_CONFLICT_NONE; // Flip for next
849-
bool fixedBank[3];
850849

851850
for (int i = 0; i < 3; i += 1) {
852851
dcls[i] = nullptr;
853852
opndDcls[i] = nullptr;
854-
fixedBank[i] = false;
855853

856854
G4_Operand *src = inst->getSrc(i);
857855
if (!src || !src->isSrcRegRegion() || src->isAreg()) {
@@ -865,15 +863,6 @@ void BankConflictPass::setupBankConflictsforMad(G4_INST *inst) {
865863
gra.kernel.numEltPerGRF<Type_UB>();
866864
srcBC[i] = gra.getBankConflict(dcls[i]);
867865

868-
if (dcls[i]->getRegVar() &&
869-
dcls[i]->getRegVar()->isPhyRegAssigned()) {
870-
int regNum = dcls[i]->getRegVar()->getPhyReg()->asGreg()->getRegNum();
871-
srcBC[i] = regNum % 2 ? BANK_CONFLICT_SECOND_HALF_ODD
872-
: BANK_CONFLICT_FIRST_HALF_EVEN;
873-
gra.setBankConflict(dcls[i], srcBC[i]);
874-
fixedBank[i] = true;
875-
}
876-
877866
if (srcBC[i] != BANK_CONFLICT_NONE) {
878867
if (isOddOffset(offset[i])) {
879868
if (srcBC[i] == BANK_CONFLICT_FIRST_HALF_EVEN) {
@@ -927,7 +916,7 @@ void BankConflictPass::setupBankConflictsforMad(G4_INST *inst) {
927916
}
928917

929918
srcBC[i] = gra.getBankConflict(dcls[i]);
930-
if (!fixedBank[i] && srcBC[i] != BANK_CONFLICT_NONE) {
919+
if (srcBC[i] != BANK_CONFLICT_NONE) {
931920
if (isOddOffset(offset[i])) {
932921
if (srcBC[i] == BANK_CONFLICT_FIRST_HALF_EVEN) {
933922
srcBC[i] = BANK_CONFLICT_SECOND_HALF_ODD;
@@ -954,7 +943,7 @@ void BankConflictPass::setupBankConflictsforMad(G4_INST *inst) {
954943
: BANK_CONFLICT_FIRST_HALF_EVEN;
955944
}
956945
gra.setBankConflict(dcls[i], srcBC[i]);
957-
} else if (!fixedBank[i]) {
946+
} else {
958947
srcBC[i] = (assignedBank == BANK_CONFLICT_FIRST_HALF_EVEN)
959948
? BANK_CONFLICT_SECOND_HALF_ODD
960949
: BANK_CONFLICT_FIRST_HALF_EVEN;

0 commit comments

Comments
 (0)