Some llvm instruction visitors require llvm_unreachable function.

Especially those who do not generate code and do not return true.

Change-Id: I98949feb68831f6e80645458cea88055a04c60aa

Author: lukaszgotszaldintel

IGC/Compiler/CISACodeGen/Emu64OpsPass.cpp

@@ -664,16 +664,17 @@ bool InstExpander::visitInstruction(Instruction& I) {
 #if 1
     errs() << "I = " << I << '\n';
 #endif
-    IGC_ASSERT_EXIT_MESSAGE(0, "UNKNOWN INSTRUCTION is BEING EXPANDED!");
+    llvm_unreachable("UNKNOWN INSTRUCTION is BEING EXPANDED!");
     return false;
 }
 
 bool InstExpander::visitRet(ReturnInst& RI) {
-    if(Value * V = RI.getReturnValue())
+    if (Value * V = RI.getReturnValue())
     {
         // TODO: Add 64-bit return value support when function/subroutine call is supported.
         IGC_ASSERT(nullptr != Emu);
-        IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(V), "TODO: NOT IMPLEMENTED YET!");
+        if (Emu->isInt64(V))
+            llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     }
     return false;
 }
@@ -743,25 +744,29 @@ bool InstExpander::visitMul(BinaryOperator& BinOp) {
 
 bool InstExpander::visitSDiv(BinaryOperator& BinOp) {
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&BinOp), "There should not be `sdiv` which is already emulated by library call.");
+    if (Emu->isInt64(&BinOp))
+        llvm_unreachable("There should not be `sdiv` which is already emulated by library call.");
     return false;
 }
 
 bool InstExpander::visitUDiv(BinaryOperator& BinOp) {
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&BinOp), "There should not be `udiv` which is already emulated by library call.");
+    if (Emu->isInt64(&BinOp))
+        llvm_unreachable("There should not be `udiv` which is already emulated by library call.");
     return false;
 }
 
 bool InstExpander::visitSRem(BinaryOperator& BinOp) {
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&BinOp), "There should not be `srem` which is already emulated by library call.");
+    if (Emu->isInt64(&BinOp))
+        llvm_unreachable("There should not be `srem` which is already emulated by library call.");
     return false;
 }
 
 bool InstExpander::visitURem(BinaryOperator& BinOp) {
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&BinOp), "There should not be `urem` which is already emulated by library call.");
+    if (Emu->isInt64(&BinOp))
+        llvm_unreachable("There should not be `urem` which is already emulated by library call.");
     return false;
 }
 
@@ -1226,14 +1231,15 @@ bool InstExpander::visitAtomicCmpXchg(AtomicCmpXchgInst& ACXI) {
     Value* V = ACXI.getCompareOperand();
     IGC_ASSERT(nullptr != V);
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(V), "TODO: NOT IMPLEMENTED YET!");
-
+    if (Emu->isInt64(V))
+        llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }
 
 bool InstExpander::visitAtomicRMW(AtomicRMWInst& RMW) {
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&RMW), "TODO: NOT IMPLEMENTED YET!");
+    if (Emu->isInt64(&RMW))
+        llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }
 
@@ -2001,7 +2007,7 @@ bool InstExpander::visitCall(CallInst& Call) {
     }
 
     // TODO: Add i64 emulation support.
-    IGC_ASSERT_EXIT_MESSAGE(0, "TODO: NOT IMPLEMENTED YET!");
+    llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }
 
@@ -2027,7 +2033,8 @@ bool InstExpander::visitSelect(SelectInst& SI) {
 bool InstExpander::visitVAArg(VAArgInst& VAAI) {
     // TODO: Add i64 emulation support.
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&VAAI), "TODO: NOT IMPLEMENTED YET!");
+    if (Emu->isInt64(&VAAI))
+        llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }
 
@@ -2103,22 +2110,25 @@ bool InstExpander::visitInsertElement(InsertElementInst& IEI) {
 bool InstExpander::visitExtractValue(ExtractValueInst& EVI) {
     // TODO: Add i64 emulation support.
     IGC_ASSERT(nullptr != Emu);
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(&EVI), "TODO: NOT IMPLEMENTED YET!");
+    if (Emu->isInt64(&EVI))
+        llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }
 
 bool InstExpander::visitInsertValue(InsertValueInst& IVI) {
     // TODO: Add i64 emulation support.
     IGC_ASSERT(nullptr != Emu);
     IGC_ASSERT(0 < IVI.getNumOperands());
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(IVI.getOperand(1)), "TODO: NOT IMPLEMENTED YET!");
+    if (Emu->isInt64(IVI.getOperand(1)))
+        llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }
 
 bool InstExpander::visitLandingPad(LandingPadInst& LPI) {
     // TODO: Add i64 emulation support.
     IGC_ASSERT(nullptr != Emu);
     IGC_ASSERT(0 < LPI.getNumOperands());
-    IGC_ASSERT_EXIT_MESSAGE(false == Emu->isInt64(LPI.getOperand(1)), "TODO: NOT IMPLEMENTED YET!");
+    if (Emu->isInt64(LPI.getOperand(1)))
+        llvm_unreachable("TODO: NOT IMPLEMENTED YET!");
     return false;
 }