You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Not sure if uncommenting these (and other lines in generateStencilClasses()) enables support for globals right away.
(thanks to Joerg Behrens for reporting this)
The text was updated successfully, but these errors were encountered:
## Technical Description
This PR enables globals for the unstructured backends. Furthermore, an unreported issue where globals were not propagated from the wrapper class to the stencil class was fixed. Additionally, a bug in the unstructured cuda codegen was fixed when translating stencils that only use dense dimensions.
### Resolves / Enhances
Fixes#1030Fixes#1028
### Notes
The methods to set and get globals in the cuda backend are on the inner stencils. This will be addressed in [this issue](#1038). Also, a method to communicate globals from FORTRAN will need to be devised (not addressed yet).
### Testing
New tests in dawn4py and a new unstructured integration test to test the correct operation of the `CXXNaiveIco` backend. `CudaIco` backend tested manually.
### Dependencies
This PR is independent.
Support for globals is still not enabled in
CXXNaiveIcoCodeGen
, see:dawn/dawn/src/dawn/CodeGen/CXXNaive-ico/CXXNaiveCodeGen.cpp
Lines 382 to 385 in aa257ec
Not sure if uncommenting these (and other lines in
generateStencilClasses()
) enables support for globals right away.(thanks to Joerg Behrens for reporting this)
The text was updated successfully, but these errors were encountered: