-
Notifications
You must be signed in to change notification settings - Fork 12
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
Skip privatization of arrays with existing data declarations #389
Conversation
Documentation for this branch can be viewed at https://sites.ecmwf.int/docs/loki/389/index.html |
cf89b7c
to
cf0f893
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #389 +/- ##
==========================================
- Coverage 95.57% 95.57% -0.01%
==========================================
Files 201 201
Lines 39810 39833 +23
==========================================
+ Hits 38050 38071 +21
- Misses 1760 1762 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
e01d916
to
3ee37ed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, very nice addition and implementation is well structured and tested.
This requires a rebase and a few small-ish comments that would be good to take care of.
@@ -221,14 +266,16 @@ def device_alloc_column_locals(cls, routine, column_locals): | |||
routine.body.prepend((ir.Comment(''), pragma, ir.Comment(''))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[no action required] This is not covered in tests, so we don't seem to be testing this.
(@mlange05 for info)
3ee37ed
to
dbbf717
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks, changes look good to me!
…find_driver_loops and annotate_driver_loop
dbbf717
to
66215b6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, GTG.
Arrays with existing data declarations should not be privatized in the driver gang loop. This PR adds the functionality to gather information from encompassing
!$acc data
regions and filter out the relevant arrays.