[Mlir-commits] [mlir] [MLIR] Fix a problem with parseOptionalAttribute (PR #188125)

[Mehdi Amini]

joker-eph wrote:

I looked into the test case, before it was failing with:

```
error: custom op 'test.format_opt_attrdef_before_symbol' expected valid '@'-identifier for symbol name
```

And now it is failing with:

```
error: custom op 'test.format_opt_attrdef_before_symbol' expected attribute of a different type
```

It's not clear to me why the first one would be better here?

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