[Mlir-commits] [mlir] [MLIR][LLVM] Add bytecode support for several attributes (PR #162577)

[Bruno Cardoso Lopes]

================
@@ -0,0 +1,81 @@
+// RUN: mlir-opt -emit-bytecode %s | mlir-opt --mlir-print-debuginfo | FileCheck %s
----------------
bcardosolopes wrote:

Changed both `mlir/test/Dialect/LLVMIR/roundtrip.mlir` and `bytecode.mlir` to use `-verify-roundtrip`. `mlir/test/Dialect/LLVMIR/debuginfo.mlir` fails with `-verify-roundtrip` even before this PR, so there's technical debt that needs to be fixed there first.

I also had to remove the support for `AliasScopeAttr` because `DistinctAttr` needs some especial handling to allow roundtripping, the existing textual form in the bytecode doesn't roundtrip either.

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