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.

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

fix Deallocate printer #8

Open
bauom opened this issue Dec 5, 2022 · 1 comment · Fixed by pyccel/pyccel#1266
Open

fix Deallocate printer #8

bauom opened this issue Dec 5, 2022 · 1 comment · Fixed by pyccel/pyccel#1266
Assignees
@framdani
Labels
invalid This doesn't seem right

Comments

@bauom
Copy link
Collaborator

bauom commented Dec 5, 2022 •
edited by framdani
Loading

Fix the printer of deallocate to free the space allocated by one of the cuda's memory allocation APIs.
@Pinkyboi
Copy link
Member

Pinkyboi commented Dec 5, 2022 •
edited
Loading

@bauom can you be more precise as to what needs to be fixed in the deallocate printer ?

@framdani framdani self-assigned this Dec 19, 2022
@EmilyBourne EmilyBourne transferred this issue from pyccel/pyccel Mar 14, 2023
@EmilyBourne EmilyBourne mentioned this issue Mar 21, 2023
Merged
@EmilyBourne EmilyBourne linked a pull request Mar 22, 2023 that will close this issue
Fix deallocate printer pyccel/pyccel#1266
Merged
bauom added a commit to pyccel/pyccel that referenced this issue Apr 19, 2023
@framdani @EmilyBourne @bauom
Fix deallocate printer (#1266)
0bbad8f 
Update the printer of deallocate to free the array object allocated by
one of the cuda's memory allocation APIs. Fixes pyccel#8

 **Functions added:**
- `cuda_free_host` : frees the memory returned by cudaMallocHost().
- `cuda_free` : frees the array located on the device.
- `cuda_free_pointer` : frees a pointer to a cuda array object.

---------

Co-authored-by: EmilyBourne <[email protected]>
Co-authored-by: bauom <[email protected]>
@EmilyBourne EmilyBourne added the invalid This doesn't seem right label Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@framdani framdani

Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix deallocate printer pyccel/pyccel
4 participants
@EmilyBourne @bauom @Pinkyboi @framdani