[PATCH] D100575: [flang][OpenMP][FIX] Fix the worksharing nesting check with inclusion of more constructs to cover entire workshare family.

Arnamoy B via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 15 09:17:11 PDT 2021


arnamoy10 created this revision.
arnamoy10 added reviewers: kiranchandramohan, bryanpkc, kiranktp, AMDChirag, clementval.
arnamoy10 added projects: Flang, OpenMP.
Herald added subscribers: jdoerfert, jfb, guansong, yaxunl.
arnamoy10 requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a project: LLVM.

Existing code for checking the nesting of workshare regions was not covering all the possible constructs that can be workshare constructs.  This patch creates a new group of constructs for workshare constructs.

It extends the check to check during sections constructs as well.

Also adds a few test cases to verify the check.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D100575

Files:
  flang/lib/Semantics/check-omp-structure.cpp
  flang/lib/Semantics/check-omp-structure.h
  flang/test/Semantics/omp-do05.f90
  flang/test/Semantics/omp-workshare01.f90
  flang/test/Semantics/omp-workshare04.f90
  flang/test/Semantics/omp-workshare05.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100575.337787.patch
Type: text/x-patch
Size: 8004 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210415/67082328/attachment.bin>


More information about the llvm-commits mailing list