[Mlir-commits] [mlir] [mlir][OpenMP] implement SIMD reduction (PR #146671)

[Sergio Afonso]

================
@@ -2872,11 +2883,27 @@ convertOmpSimd(Operation &opInst, llvm::IRBuilderBase &builder,
   if (handleError(afterAllocas, opInst).failed())
     return failure();
 
+  if (failed(allocReductionVars(simdOp, reductionArgs, builder,
+                                moduleTranslation, allocaIP, reductionDecls,
+                                privateReductionVariables, reductionVariableMap,
+                                deferredStores, isByRef)))
+    return failure();
+
   if (handleError(initPrivateVars(builder, moduleTranslation, privateVarsInfo),
                   opInst)
           .failed())
     return failure();
 
+  // TODO: no call to copyFirstPrivateVars?
----------------
skatrak wrote:

That seems to be missing indeed. I'm guessing `firstprivate` probably doesn't currently work for `simd`, then.

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