[flang-commits] [PATCH] D93105: [Flang][openmp][5.0] Add task_reduction clause.

[Valentin Clement via Phabricator via flang-commits]

clementval added inline comments.


================
Comment at: flang/lib/Parser/openmp-parsers.cpp:223
+        construct<OmpClause>(construct<OmpClause::TaskReduction>(
+            parenthesized(Parser<OmpReductionClause>{}))) ||
     "RELAXED" >> construct<OmpClause>(construct<OmpClause::Relaxed>()) ||
----------------
clementval wrote:
> @sameeranjoshi After our slack discussion, I hope you don't mind but I updated your patch with a working version without the indirection. 
> 
> The reduction clause is actually not done "right" at the moment since it doesn't follow other clauses. It should be done in a similar way as `TASK_REDUCTION` now. I guess we can do this in a separated patch. 
update to reduction clause are done in D93482


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93105/new/

https://reviews.llvm.org/D93105