[PATCH] D91871: [flang][openacc] Add clause validity tests for the parallel directive
Valentin Clement via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 20 10:22:21 PST 2020
clementval added a comment.
In D91871#2408413 <https://reviews.llvm.org/D91871#2408413>, @sameeranjoshi wrote:
> IIUC, the rational for adding these tests was to just add clauses which might be missing from the list of `AccClause` ?
I'm currently passing on all directive and make sure that all accepted clauses are actually accepted and restrictions are in place correctly. In this case, ACC.td and the sema restrictions were correct already but just wanted to make sure to add those tests to avoid future regression.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91871/new/
https://reviews.llvm.org/D91871
More information about the llvm-commits
mailing list