Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Closes #1664) Add ACCWaitDirective and async keyword on kernels, enter data, parallel #2664

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

arporter
Copy link
Member

This is a replacement/takeover of #2070 as Sebastien has moved to a new job.

@arporter
Copy link
Member Author

Now that I look at this again, there don't seem to be any tests of the new directive and keyword for the Fortran psyir backend (it's all gen_code in LFRic).

@arporter
Copy link
Member Author

I see from #2070 (comment) that this PR was started before we had ACCClause (#2157). However, we do now have clauses and therefore this code needs updating (to remove the use of Signature) and tests for the Fortran backend are required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants