[Mlir-commits] [mlir] [mlir][sparse] Expand LevelType to 64 bits and implement parsing n out of m (PR #79935)

Yinying Li llvmlistbot at llvm.org
Fri Feb 2 11:36:43 PST 2024


================
@@ -229,54 +263,51 @@ constexpr const char *toMLIRString(LevelType lt) {
     return "loose_compressed(nonordered)";
   case LevelType::LooseCompressedNuNo:
     return "loose_compressed(nonunique, nonordered)";
-  case LevelType::TwoOutOfFour:
-    return "block2_4";
+  default:
----------------
yinying-lisa-li wrote:

It would not catch n:m with n and m set. I could do `auto lt = static_cast<LevelType>(static_cast<uint64_t>(lvlType) & 0xffffffff);` and use `case LevelType::NOutOfM`. But since there's a function isNOutOfM, I chose that for readability. But I don't have a strong preference, so if you prefer having a simple case, I can do that as well.

https://github.com/llvm/llvm-project/pull/79935


More information about the Mlir-commits mailing list