================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to