[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