-
Notifications
You must be signed in to change notification settings - Fork 38
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
Permit INV_TRANS with OpenMP offload #196
base: develop
Are you sure you want to change the base?
Conversation
c14da12
to
0a390d0
Compare
Current state of play: the test program crashes in !$OMP TARGET DATA USE_DEVICE_PTR(ZAA,ZINP,ZOUTA) with error
The similar statement above for the 0-mode double-precision arrays ( |
After further investigation, it seems that |
b2014c1
to
f25d9ea
Compare
These variables should be inherited from the parent scope and shared between all threads. This was a bug.
Otherwise this will not compile with CCE and -acc.
We already put this in LEINV, so why is it needed? No idea.
Co-authored-by: Paul Mullowney <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Sam Hatfield <[email protected]>
Co-authored-by: Thomas Gibson <[email protected]>
e632882
to
ff82d59
Compare
#210 should be merged before this. |
Now that we can compile with OpenMP offload, I will gradually add back this capability to ecTrans until we can run the benchmark program.
The purpose of this PR is to allow, at least in a minimal fashion, OpenMP offload to be used with the inverse transform. This PR can be merged once this program produces the correct behaviour:
This just performs an inverse transform on a single 2D scalar field of zeroes.