[llvm-branch-commits] [mlir] [MLIR][OpenMP] Update omp.wsloop translation to LLVM IR (4/5) (PR #89214)

[Sergio Afonso via llvm-branch-commits]

================
@@ -916,49 +916,50 @@ static LogicalResult inlineReductionCleanup(
 static LogicalResult
 convertOmpWsloop(Operation &opInst, llvm::IRBuilderBase &builder,
                  LLVM::ModuleTranslation &moduleTranslation) {
-  auto loop = cast<omp::WsloopOp>(opInst);
-  const bool isByRef = loop.getByref();
+  auto wsloopOp = cast<omp::WsloopOp>(opInst);
+  auto loopOp = cast<omp::LoopNestOp>(wsloopOp.getWrappedLoop());
----------------
skatrak wrote:

Yes, this won't work for composite constructs. The current verifiers allow the composite constructs that are supported by the spec, although they aren't supported by Flang lowering or by the translation from MLIR to LLVM IR yet. That's something that will come later on, the purpose of this patch is just to keep everything that worked before working after the changes to `omp.wsloop`.

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