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

sameeran joshi via Phabricator via flang-commits flang-commits at lists.llvm.org
Mon Dec 14 11:13:00 PST 2020


sameeranjoshi added inline comments.


================
Comment at: flang/include/flang/Parser/parse-tree.h:3430
+  TUPLE_CLASS_BOILERPLATE(OmpTaskReductionClause);
+  std::tuple<OmpReductionOperator, OmpObjectList> t;
+};
----------------
clementval wrote:
> Is there a good reason why `OmpReductionClause` takes a list of `Designator` and here you take an `OmpObjectList`? 
I did ask once the same in slack channel.
But I think the complicated checks can be done in semantic part given the tablegen framework.
https://flang-compiler.slack.com/archives/CGB8G85QV/p1603382608040500


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93105



More information about the flang-commits mailing list