Andres-Salamanca wrote: @erichkeane yeah MLIR doesn't have it natively, the closest is `oilist` but it doesn't support comma separators. the issue is the comma after the name e.g. `["n", init]` dropping it would break all existing tests. i'm not a big fan of the custom parser either, i just created this PR because i was blocked on another one 😅 if dropping the commas is fine i can go ahead and update all the tests and use `oilist`, or we could also go with what @xlauko suggested and add `comma-separated-oilist` to MLIR.
https://github.com/llvm/llvm-project/pull/198962 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
