Skip to content

[flang][cuda] Allocate the dst descriptor in data transfer #143437

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 10, 2025
Merged

[flang][cuda] Allocate the dst descriptor in data transfer #143437

merged 1 commit into from
Jun 10, 2025

Conversation

clementval
Copy link
Contributor

In a test like:

integer, allocatable, device :: da(:)
allocate(a(200))
a = 2
da = a ! da is not allocated before data transfer is initiated. Allocate it with a

The reference compiler will allocate the data for the da descriptor so the data transfer can be done properly.

@clementval clementval requested a review from wangzpgi June 9, 2025 20:37
@clementval clementval merged commit 9c54512 into llvm:main Jun 10, 2025
8 checks passed
@clementval clementval deleted the cuf_desc_desc_alloc branch June 10, 2025 16:43
rorth pushed a commit to rorth/llvm-project that referenced this pull request Jun 11, 2025
@clementval @rorth
[flang][cuda] Allocate the dst descriptor in data transfer (llvm#143437)
5831a21 
In a test like: 

```
integer, allocatable, device :: da(:)
allocate(a(200))
a = 2
da = a ! da is not allocated before data transfer is initiated. Allocate it with a
```

The reference compiler will allocate the data for the `da` descriptor so
the data transfer can be done properly.
tomtor pushed a commit to tomtor/llvm-project that referenced this pull request Jun 14, 2025
@clementval @tomtor
[flang][cuda] Allocate the dst descriptor in data transfer (llvm#143437)
f2ca1c9 
In a test like: 

```
integer, allocatable, device :: da(:)
allocate(a(200))
a = 2
da = a ! da is not allocated before data transfer is initiated. Allocate it with a
```

The reference compiler will allocate the data for the `da` descriptor so
the data transfer can be done properly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wangzpgi wangzpgi wangzpgi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@clementval @wangzpgi