[PATCH] D91873: [flang][openacc] Add clause validity tests for the kernels directive

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 4 07:52:52 PST 2020


clementval added a comment.

Thanks for the review!



================
Comment at: flang/test/Semantics/acc-clause-validity.f90:529
+
+  !$acc kernels default(present)
+  !$acc end kernels
----------------
sameeranjoshi wrote:
> When default has value other than `none` or `present` ?
It will be a syntax error caught by the parser. I'll add a test to ensure that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91873



More information about the llvm-commits mailing list