[Mlir-commits] [mlir] [mlir][OpenMP] inscan reduction modifier and scan op mlir support (PR #114737)

[Sergio Afonso]

================
@@ -455,6 +523,7 @@ class OpenMP_InReductionClauseSkip<
   ];
 
   let arguments = (ins
+    OptionalAttr<ReductionModifierAttr>:$in_reduction_mod,
----------------
skatrak wrote:

I may be misunderstanding your concern, so let me know if I am, but are you referring to the _reduction-identifier_ in the _Modifiers_ table? That would be e.g. the '+' in `reduction(+: acc)`, which is indirectly represented by the `*_syms` operands.

The _reduction-modifier_ that you're introducing a new operand for is actually only present in the _Modifiers_ table for `reduction`, as far as I can tell.

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