[Mlir-commits] [mlir] [mlir][docs] Update Bytecode documentation (PR #99854)

[Jacques Pienaar]

================
@@ -172,31 +172,37 @@ dialects that were also referenced.
 ```
 dialect_section {
   numDialects: varint,
-  dialectNames: varint[],
-  numTotalOpNames: varint,
-  opNames: op_name_group[]
+  dialectNames: dialect_name_group[],
+  opNames: dialect_ops_group[]  // ops grouped by dialect
 }
 
-op_name_group {
-  dialect: varint // (dialectID << 1) | (hasVersion),
-  version : dialect_version_section
-  numOpNames: varint,
-  opNames: varint[]
+dialect_name_group {
+  nameAndIsVersioned: varint  // (dialectID << 1) | (hasVersion),
+  version: dialect_version_section  // only if versioned
 }
 
 dialect_version_section {
   size: varint,
   version: byte[]
 }
 
+dialect_ops_group {
+  dialect: varint,
+  numOpNames: varint,
----------------
jpienaar wrote:

Isn't this implicit in the way we write here? (e.g., above we have an array but we don't say number of elements in array are materialized, that's just how all arrays are)

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