From: Nathaniel McVicar <namcvica@microsoft.com>
Date: Tue, 4 Oct 2022 16:42:43 +0000 (-0700)
Subject: [mlir][sparse] Restore case coverage warning fix
X-Git-Tag: upstream/17.0.6~31643
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=83839700c32996c58ddebc0c74e3dc4970e005bc;p=platform%2Fupstream%2Fllvm.git

[mlir][sparse] Restore case coverage warning fix

This restores the fix from D134925 to make MSVC and clang happy.

Reviewed By: stella.stamenova

Differential Revision: https://reviews.llvm.org/D135126
---

diff --git a/mlir/lib/ExecutionEngine/SparseTensor/File.cpp b/mlir/lib/ExecutionEngine/SparseTensor/File.cpp
index eac3b02..b263231 100644
--- a/mlir/lib/ExecutionEngine/SparseTensor/File.cpp
+++ b/mlir/lib/ExecutionEngine/SparseTensor/File.cpp
@@ -24,6 +24,7 @@
 //===----------------------------------------------------------------------===//
 
 #include "mlir/ExecutionEngine/SparseTensor/File.h"
+#include "llvm/Support/ErrorHandling.h"
 
 #include <cctype>
 #include <cstring>
@@ -108,6 +109,7 @@ bool SparseTensorFile::canReadAs(PrimaryType valTy) const {
     // integer and floating primary-types.
     return isRealPrimaryType(valTy);
   }
+  llvm_unreachable("unknown ValueKind");
 }
 
 /// Helper to convert C-style strings (i.e., '\0' terminated) to lower case.