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.

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

fix memory leak in gpu simulator #717

Merged
merged 1 commit into from
Apr 27, 2020
Merged

fix memory leak in gpu simulator #717

merged 1 commit into from
Apr 27, 2020

Conversation

hhorii
Copy link
Collaborator

@hhorii hhorii commented Apr 26, 2020

Summary

Fix a bug to leak GPU memory in qubitvector_thrust.hpp.

Details and comments

An abstract class to manage host or device memory does not have virtual for its desconstructor.

This fix will resolve #662.

@atilag atilag merged commit a297570 into Qiskit:master Apr 27, 2020
@chriseclectic chriseclectic added the bug Something isn't working label May 1, 2020
@chriseclectic chriseclectic added the stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable label May 29, 2020
@chriseclectic chriseclectic added this to the 0.5.2 milestone May 29, 2020
@chriseclectic chriseclectic added Changelog: Bugfix Include in the Fixed section of the changelog and removed bug Something isn't working labels May 29, 2020
chriseclectic pushed a commit to chriseclectic/qiskit-aer that referenced this pull request May 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: Bugfix Include in the Fixed section of the changelog stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using GPU causes Data to not be found
4 participants