[PATCH] D108512: [flang][OpenMP] Add semantic checks for ordered construct

Peixin Qiao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 3 00:24:30 PDT 2021


peixin updated this revision to Diff 370492.
peixin added a comment.

Replace `GetContext.clauseSource()` with `itr->second->source` since `GetContext.clauseSource()` points the last clause in the clauselist while `itr->second->source` points the clause being checked. This fix makes the informative error pointing to the right code.


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

https://reviews.llvm.org/D108512

Files:
  flang/lib/Semantics/check-omp-structure.cpp
  flang/lib/Semantics/check-omp-structure.h
  flang/lib/Semantics/resolve-directives.cpp
  flang/test/Semantics/omp-clause-validity01.f90
  flang/test/Semantics/omp-ordered01.f90
  llvm/include/llvm/Frontend/OpenMP/OMP.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D108512.370492.patch
Type: text/x-patch
Size: 11071 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210903/353603d4/attachment.bin>


More information about the llvm-commits mailing list