[PATCH] D112217: [flang][flang-omp-report] Remove the loop workarounds for nowait clause

Josh Mottley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 21 05:27:22 PDT 2021


josh.mottley.arm created this revision.
Herald added a reviewer: sscalpone.
Herald added a project: Flang.
josh.mottley.arm requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, sstefan1, jdoerfert.
Herald added a project: LLVM.

In a "Worksharing-loop construct", one can add a nowait clause at the end
of the loop (i.e. at the loop tail). This clause wasn't associated with
the corresponding loop when originally worked on in flang-omp-report.
Note that this refers to parsing and parse-tree generation. To work
around it, it was needed to track such clauses and the loops.
This should no longer be required (and in fact no longer works) and so
was removed. This results in 'curLoopLogRecord' and 'loopLogRecordStack' and
all references to them being removed. This also allows for
'constructClauses' to be swapped from std::deque to llvm::SmallVector.
Lastly a new test has been added testing the "nowait" clauses in a
variety of ways.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D112217

Files:
  flang/examples/flang-omp-report-plugin/flang-omp-report-visitor.cpp
  flang/examples/flang-omp-report-plugin/flang-omp-report-visitor.h
  flang/examples/flang-omp-report-plugin/flang-omp-report.cpp
  flang/test/Examples/omp-nowait.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112217.381225.patch
Type: text/x-patch
Size: 15109 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211021/da9f4ee1/attachment.bin>


More information about the llvm-commits mailing list