Skip to content

Commit 11f2f86

Browse files
DianaChenigcbot
authored andcommitted
IGA SWSB: consider illegal instructions in the input
In SWSB setter, skip illeagl instruction. Illegal instruction cannot have dependency to others, nor does it affect the in-pipe instruction count. Skipping it can avoid we accidentally set SWSB on it (e.g. when forcing B2B dependency). Illeagl instructions cannot carry SWSB info.
1 parent 3881a36 commit 11f2f86

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

visa/iga/IGALibrary/IR/SWSBSetter.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1136,6 +1136,12 @@ void SWSBAnalyzer::run() {
11361136
++instIter) {
11371137
m_InstIdCounter.global++;
11381138
inst = *instIter;
1139+
// skip illeagl instruction. It cannot have dependency to others, nor does
1140+
// it affect the in-pipe instruction count. Skipping it can avoid we
1141+
// accidentally set SWSB on it (e.g. when forcing B2B dependency). Illeagl
1142+
// instructions cannot carry SWSB info.
1143+
if (inst->getOp() == Op::ILLEGAL)
1144+
continue;
11391145
DepSet *input = nullptr;
11401146
DepSet *output = nullptr;
11411147
size_t dpas_cnt_in_macro = 0;

0 commit comments

Comments
 (0)