[Mlir-commits] [mlir] [mlirbc] Add missing encoding for float types (PR #191962)

[Jacques Pienaar]

================
@@ -231,6 +231,12 @@ def BFloat16Type : DialectType<(type)>;
 
 def Float16Type : DialectType<(type)>;
 
+// Stage the addition of new floating point types so that readers can be updated
+// first.
----------------
jpienaar wrote:

It is part of the RFC and we have done this out of tree before for types & version, and in tree with version. This is enabling the reader but not the writer so that it gives time where it doesn't break folks so that the reader binaries can be rolled out (making lives easier of folks integrating changes). Could also update the serialized file using the writer enabled, then the reader is tested.

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