[Mlir-commits] [mlir] [mlir][vector] Fix round-trip parsing of operand-less vector.print (PR #231851)

[Matthias Springer]

https://github.com/matthias-springer commented:

I find it a bit surprising that both `vector.print` and `vector.print punctuation <newline>` print a new-line character. What does `punctuation <newline>` even mean in that case?

There's another way to fix the issue: require that operands are wrapped in parentheses. E.g. `vector.print (%x, %y) : i32, i32`. However, that will require changes to existing IR.

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