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 fusion CostBasedFusion::aggregate_operations #2075

Merged
merged 4 commits into from
Mar 7, 2024

Conversation

doichanj
Copy link
Collaborator

Summary

This is fix for #2074

Details and comments

fixes overrun of cost array

@doichanj doichanj added stable-backport-potential The issue or PR might be minimal and/or import enough to backport to stable Changelog: Bugfix Include in the Fixed section of the changelog labels Feb 27, 2024
@aromanro
Copy link
Contributor

If I recall correctly the code, j cannot get lower than fusion_start, so a check for inequality is of no use. == should be a little bit faster. Not that it matters much.

hhorii
hhorii previously approved these changes Feb 29, 2024
@doichanj doichanj merged commit 76b1042 into Qiskit:main Mar 7, 2024
36 checks passed
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.

3 participants