[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