[PATCH] D89546: [Flang][OpenMP 4.5] Add semantic check for OpenMP schedule clause

Fri Oct 23 05:58:44 PDT 2020

yhegde updated this revision to Diff 300255.
yhegde added a comment.

1. Three more test cases are added.
2. RequiresPositiveParameter() function is not called, because with the custom implementation  we can give specific error message pointing to invalid chunk size.  If we use that function we will end up in throwing generic error message.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89546

Files:
  flang/lib/Semantics/check-omp-structure.cpp
  flang/test/Semantics/omp-do-schedule.f90
  flang/test/Semantics/omp-do-schedule02.f90
  flang/test/Semantics/omp-do-schedule03.f90
  flang/test/Semantics/omp-do-schedule04.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D89546.300255.patch
Type: text/x-patch
Size: 4251 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201023/d0b2a54e/attachment.bin>