Skip to content

Add unroll step to level2 passmanager optimization loop (bp #5671)#5683

Merged
mergify[bot] merged 1 commit into
stable/0.16from
mergify/bp/stable/0.16/pr-5671
Jan 22, 2021
Merged

Add unroll step to level2 passmanager optimization loop (bp #5671)#5683
mergify[bot] merged 1 commit into
stable/0.16from
mergify/bp/stable/0.16/pr-5671

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify Bot commented Jan 22, 2021

This is an automatic backport of pull request #5671 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.io/

This commit adds the basis translation step to the level 2 preset
passmanager's optimization loop. The level 2 optimization loop runs the
commutative cancellation pass which use Rx and U1 gates as part of it's
optimization step which may not be in the target basis gates. This can
result in a gate outside of the target basis ended up in the output. To
avoid this in a general manner we need to rely on the basis translation
step because we can't write a pass to do the simplified rotation in a
manner compatible with any basis (especially since the target basis
could be using custom gates).

Fixes #5644

(cherry picked from commit 25eb58a)
@mergify mergify Bot requested a review from a team as a code owner January 22, 2021 18:42
@mtreinish mtreinish mentioned this pull request Jan 22, 2021
6 tasks
@kdk kdk added the automerge label Jan 22, 2021
@mergify mergify Bot merged commit 962fb03 into stable/0.16 Jan 22, 2021
@mergify mergify Bot deleted the mergify/bp/stable/0.16/pr-5671 branch January 22, 2021 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants