[mlir][sparse] Case coverage fix no errorhandling

[NathanielMcVicar]

Summary

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

[aartbik]

mlir/lib/ExecutionEngine/SparseTensor/File.cpp

111

I believe Wren was already taking care of that in https://reviews.llvm.org/D135186
That would also roll back the bazel change, which is pulling in too much deps right now

Can I kindly request to wait for reviews going forward on this?

[stella.stamenova]

mlir/lib/ExecutionEngine/SparseTensor/File.cpp

111

We're happy to wait as long as we don't keep getting broken. @vitalybuka reverted Nate's change from yesterday today putting the windows mlir bot in the red again. We'd love to get to a solution that works for everyone, but if the changes keep getting reverted with no communication befor or after and getting the bot broken, it's really disruptive and we definitely don't want to be in a place where anyone is broken for multiple days again.

[aartbik]

mlir/lib/ExecutionEngine/SparseTensor/File.cpp

111

Ah, so this was needed again to fix breakage, I missed that completely? In that case, that makes sense of course, and you did of course the right thing. I really lost track of who is reverting and resubmitting this seemingly similar change ;-) [and earlier had to address a blaze breakage as a result].

So is all green now? In that case, we will proceed with the cosmetic follow ups ....

[stella.stamenova]

mlir/lib/ExecutionEngine/SparseTensor/File.cpp

111

Yes, the windows mlir buildbot is green again. I hope our change did not break anyone else either.

mlir/lib/ExecutionEngine/SparseTensor/File.cpp

case ValueKind::kComplex:
     // require a complex primary-type.
     return isComplexPrimaryType(valTy);
   case ValueKind::kUndefined:
     // The "extended" FROSTT format doesn't specify a ValueKind.
     // So we allow implicitly converting the stored values to both
     // 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.
 static inline void toLower(char *token) {
   for (char *c = token; *c; ++c)
     *c = tolower(*c);
 }