[Mlir-commits] [mlir] [MLIR] Add two-phase region parsing API to OpAsmParser (PR #190545)

[Mehdi Amini]

joker-eph wrote:

> Would you suggest to generalizing this? 

Possibly, in general I'm wary of API that may be too much "ad-hoc" and/or have limited use: because then we may have an explosion of APIs that don't compose super well. So I rather spend a bit more time thinking about how to serve as many use-cases as possible. 
Here I'm not sure, I would need to survey a bit more what "generalizing" would imply, this is why I asked initially about what kind of upstream ops this would cover.

>  The nice thing I found about this was that at least you can guard yourself against wrong use of state,

What do you mean by this?

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