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

[Matthias Springer]

================
@@ -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,
----------------
matthias-springer wrote:

I think the number of elements of all arrays are always explicit in here. E.g.: `dialect_version_section.size`, `dialect_section.numDialects`, `attr_type_offset_section.num_attrs`, etc.


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