[Mlir-commits] [mlir] [mlirbc] Serialize dense elements attr i1 using packed (PR #182233)

[Matthias Springer]

================
@@ -465,6 +465,14 @@ class DialectWriter : public DialectBytecodeWriter {
         "dialect blob");
   }
 
+  void writeUnownedBlob(ArrayRef<char> blob) override {
+    emitter.emitVarInt(blob.size(), "dialect blob");
+    emitter.emitBytes(
+        ArrayRef<uint8_t>(reinterpret_cast<const uint8_t *>(blob.data()),
----------------
matthias-springer wrote:

I have seen a mixture of `char`, `uint8_t` and also somebody asked about `std::byte` once. When do we use which?

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