[clang-tools-extra] [mlir][OpenMP] Added omp.region operation (PR #65243)

2023-10-11 Thread via cfe-commits
shraiysh wrote: Sure I'll do that. https://github.com/llvm/llvm-project/pull/65243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [mlir][OpenMP] Added omp.region operation (PR #65243)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Can this have `omp.yield` as the terminator and be capable of yielding/returning values? https://github.com/llvm/llvm-project/pull/65243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/ma

[clang-tools-extra] [mlir][OpenMP] Added omp.region operation (PR #65243)

2023-09-04 Thread via cfe-commits
shraiysh wrote: `internal_region` sounds good. I did not want to make it highly specific, because it could be used under other constructs too. Like, `omp.sections` could have an internal region and a barrier in the future. We do not need it for others at the moment, but not making it highly sp

[clang-tools-extra] [mlir][OpenMP] Added omp.region operation (PR #65243)

2023-09-04 Thread via cfe-commits
kiranchandramohan wrote: Please add a quick RFC in the mlir discourse channel. Would it make sense to call it an internal region? Or specialize it further and call it canonical_loop_region? https://github.com/llvm/llvm-project/pull/65243 ___ cfe-comm