[Mlir-commits] [mlir] ff7a2b6 - [mlir][sparse] Case coverage fix no errorhandling

[Stella Stamenova]

Author: Nathaniel McVicar
Date: 2022-10-05T15:35:00-07:00
New Revision: ff7a2b60555a28754b9513b78c5a6b4678b6656f

URL: https://github.com/llvm/llvm-project/commit/ff7a2b60555a28754b9513b78c5a6b4678b6656f
DIFF: https://github.com/llvm/llvm-project/commit/ff7a2b60555a28754b9513b78c5a6b4678b6656f.diff

LOG: [mlir][sparse] Case coverage fix no errorhandling

Restores the fix from D134925 for MSVC without breaking cpu runner.

Differential Revision: https://reviews.llvm.org/D135304

Added: 
    

Modified: 
    mlir/lib/ExecutionEngine/SparseTensor/File.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/ExecutionEngine/SparseTensor/File.cpp b/mlir/lib/ExecutionEngine/SparseTensor/File.cpp
index eac3b0278ce39..932d49652cc17 100644
--- a/mlir/lib/ExecutionEngine/SparseTensor/File.cpp
+++ b/mlir/lib/ExecutionEngine/SparseTensor/File.cpp
@@ -108,6 +108,8 @@ bool SparseTensorFile::canReadAs(PrimaryType valTy) const {
     // integer and floating primary-types.
     return isRealPrimaryType(valTy);
   }
+  MLIR_SPARSETENSOR_FATAL("Unknown ValueKind: %d\n",
+                          static_cast<uint8_t>(valueKind_));
 }
 
 /// Helper to convert C-style strings (i.e., '\0' terminated) to lower case.