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

[Alex Zinenko via Phabricator via llvm-commits]

ftynse added a comment.

> What exactly do you expect to do with OpenMP worksharing loops on this level which is problematic with CFGs?

It's the inverse that is problematic: having loop ops where CFG is expected. I would like to avoid seeing something like

  omp.do <...> {
    omp.for %i = <...> {
      llvm.store <...>
      // other LLVM operations here
    }
  }
  // LLVM operations as CFG here

go into `mlir-translate`, which will have to outline and lower the loop during _translation_ in this case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86071