Skip to content

Commit 64d2d36

Browse files
MaciejKalinskiigcbot
authored andcommitted
Trivial refactoring of CEncoder::CarryBorrowArith()
Remove `switch` stmt from CEncoder::CarryBorrowArith().
1 parent 34f1385 commit 64d2d36

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

IGC/Compiler/CISACodeGen/CISABuilder.cpp

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2109,19 +2109,8 @@ namespace IGC
21092109
VISA_PredOpnd* predOpnd = GetFlagOperand(m_encoderState.m_flag);
21102110
VISA_Exec_Size execSize = GetAluExecSize(dst);
21112111

2112-
switch (execSize)
2113-
{
2114-
case EXEC_SIZE_1:
2115-
case EXEC_SIZE_8:
2116-
case EXEC_SIZE_16:
2117-
break;
2118-
case EXEC_SIZE_32:
2119-
IGC_ASSERT(getGRFSize() == 64);
2120-
break;
2121-
default:
2122-
IGC_ASSERT_MESSAGE(0, "Unknown execution size on carry-borrow-arith!");
2123-
break;
2124-
}
2112+
IGC_ASSERT(execSize == EXEC_SIZE_1 || execSize == EXEC_SIZE_8 || execSize == EXEC_SIZE_16 ||
2113+
(execSize == EXEC_SIZE_32 && getGRFSize() == 64));
21252114

21262115
IGC_ASSERT_MESSAGE(m_encoderState.m_dstOperand.mod == EMOD_NONE, "addc/subb doesn't support saturate");
21272116

0 commit comments

Comments
 (0)