[Mlir-commits] [mlir] [mlir] Attempt to resolve edge cases in PassPipeline textual format (PR #118877)

[Jeremy Kun]

j2kun wrote:

@christopherbate indeed, the option is defined as

```
    ListOption<"convertOps","convert-ops", "std::string",
                "comma-separated list of ops to run this pass on ">,
```

I guess it's not so important to preserve that particular syntax, if an unset string option cannot be distinguished from an option set to empty string anyway. It's a bit weird that the syntax is invalid now, and also weird that setting it to an empty list (even though it's a string option) is valid.

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