Let EOT send participate in swsb token assignment.

using same sbid in send instructions might cause a potential stall.
intel · Dec 18, 2024 · df6763c · df6763c
1 parent e4cdf12
commit df6763c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/visa/BinaryEncodingIGA.cpp b/visa/BinaryEncodingIGA.cpp
@@ -995,7 +995,7 @@ void BinaryEncodingIGA::SetSWSB(G4_INST *inst, SWSB &sw) {
   // - send has only distance     --> $0 and distance
   // This workaround can be removed once vISA doesn't produce such SWSB.
   // Currently this could happen only on EOT send.
-  if (inst->isSend() && !sw.hasBothDistAndToken() &&
+  if (inst->isSend() &&
       !sw.verify(getIGASWSBEncodeMode(), SWSB::InstType::SEND)) {
     sw.tokenType = SWSB::TokenType::SET;
     if (sw.hasDist()) {

diff --git a/visa/LocalScheduler/SWSB_G4IR.cpp b/visa/LocalScheduler/SWSB_G4IR.cpp
@@ -3033,7 +3033,7 @@ void SWSB::tokenAllocation() {
     printf("\n=======nodeID: %d, startID: %d, endID: %d\n", node->getNodeID(),
            node->getLiveStartID(), node->getLiveEndID());
 #endif
-    if (inst->isEOT()) {
+    if (!fg.builder->hasFourALUPipes() && inst->isEOT()) {
       continue;
     }