[PATCH] D97964: [flang][OpenMP] Add semantic check for occurrence of multiple list items in aligned clause for simd directive

Arnamoy B via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 8 15:27:23 PST 2021


arnamoy10 updated this revision to Diff 329154.
arnamoy10 added a comment.

Addressing reviewers' comments:

1. Use of `SymbolSet` instead of searching by variable name.
2. Change function name to  `FindClauses()`
3. One more test scenario to check `aligned(a) private(a) aligned(a)`
4. Simplified loop logic in case of Sema error


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

https://reviews.llvm.org/D97964

Files:
  flang/lib/Semantics/check-directive-structure.h
  flang/lib/Semantics/check-omp-structure.cpp
  flang/test/Semantics/omp-simd-aligned.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97964.329154.patch
Type: text/x-patch
Size: 3004 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210308/46c7435e/attachment.bin>


More information about the llvm-commits mailing list