Outdated comment and turn check+failure into assert

Author: bcardosolopes

mlir/include/mlir/Target/LLVMIR/LLVMImportInterface.h

@@ -167,8 +167,7 @@ class LLVMImportInterface
     // Lookup the dialect interface for the given intrinsic.
     // Verify the intrinsic identifier maps to an actual intrinsic.
     llvm::Intrinsic::ID intrinId = inst->getIntrinsicID();
-    if (intrinId == llvm::Intrinsic::not_intrinsic)
-      return failure();
+    assert(intrinId != llvm::Intrinsic::not_intrinsic);
 
     // First lookup intrinsic across different dialects for known
     // supported converstions, examples include arm-neon, nvm-sve, etc

mlir/lib/Target/LLVMIR/Dialect/LLVMIR/LLVMIRToLLVMTranslation.cpp

@@ -63,7 +63,6 @@ convertUnregisteredIntrinsicImpl(OpBuilder &odsBuilder, llvm::CallInst *inst,
                                  LLVM::ModuleImport &moduleImport) {
   StringRef intrinName = inst->getCalledFunction()->getName();
 
-  // Sanity check the intrinsic ID.
   SmallVector<llvm::Value *> args(inst->args());
   ArrayRef<llvm::Value *> llvmOperands(args);