[PATCH] D72223: [MLIR] Introduce affine.execute_region op

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

Re-read the RFC on "graybox", I am still entirely unconvinced by the current design and the explicit capture of memref used here. I think this is less flexible and not justified all-in-all.  believe the more extensible alternative is to support opaque nested regions in general and treat them conservatively as new scope (basically what I wrote above about reversing the trait).
But I'm not willing to die on this hill today, if @ftynse agrees with you that `affine` should stay a bit more isolated, and that this explicit capture is really worth it right now, then feel free to move forward.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72223