[PATCH] D74681: [mlir][DeclarativeParser] Add basic support for optional groups in the assembly format.

[Lei Zhang via Phabricator via llvm-commits]

antiagainst added inline comments.


================
Comment at: mlir/docs/OpDefinitions.md:655
+  // of operands.
+  let assemblyFormat = "attr-dict ($operands? `:` type($operands))";
+}
----------------
antiagainst wrote:
> It's sort of counter intuitive for me to see this. I'd assume something like `($operands` `:` type($operands))?` would be more natural. But maybe that's drawing too much from regex background. What about something like `($operands^` `:` type($operands))?` so one uses `^` to mean anchor and still retain overall similarity with regex?
Reformat:


It's sort of counter intuitive for me to see this. I'd assume something like `($operands : type($operands))?` would be more natural. But maybe that's drawing too much from regex background. What about something like `($operands^ : type($operands))?` so one uses `^` to mean anchor and still retain overall similarity with regex?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74681/new/

https://reviews.llvm.org/D74681