[PATCH] D86071: [MLIR][OpenMP] Add omp.do operation

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

In D86071#2265876 <https://reviews.llvm.org/D86071#2265876>, @ftynse wrote:

> In particular, it is not clear to me how this construct will connect to loops and what the lowering flow is. Does it expect an `scf.for`/`scf.parallel` as the only nested op? Is there a plan for a separate `omp.for`? How long do loops persist when we go to LLVM, given that OpenMPIRBuilder does not handle loop constructs and we really want to avoid converting loops to CFG during MLIR->LLVM IR translation.

The last part is unclear to me TBH. What exactly do you expect to do with OpenMP worksharing loops on this level which is problematic with CFGs?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86071