[flang-commits] [flang] [llvm] [flang][OpenMP] Implement COMBINER clause (PR #172036)

[Kareem Ergawy via flang-commits]

================
@@ -3740,46 +3711,69 @@ getReductionType(lower::AbstractConverter &converter,
   return reductionType;
 }
 
-static void genOMP(
-    lower::AbstractConverter &converter, lower::SymMap &symTable,
-    semantics::SemanticsContext &semaCtx, lower::pft::Evaluation &eval,
-    const parser::OpenMPDeclareReductionConstruct &declareReductionConstruct) {
+// Represent the reduction combiner as a clause, return reference to it.
+// If there is a "combiner" clause already present, do nothing. Otherwise
+// manufacture a combiner clause from the combiner expression on the reduction
----------------
ergawy wrote:

Would that degrage/pollute debug info in any way? I cannot put my finger on a specific issue but we are adding an OpenMP 6.0 clause even to older versions.

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