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

[Tobias Gysi]

https://github.com/gysit approved this pull request.

LGTM modulo the left over comments (plus possibly using the roundtrip flag in the test).

It seems there are some left over comments from the first review that github kindly folded away (in this file mlir/lib/Dialect/LLVMIR/IR/LLVMDialectBytecode.cpp). And maybe giving the roundtrip flag a shot makes sense as well. 


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