Update MCX synthesis to use `PhaseGate` instead of `U1Gate`

Qiskit / qiskit

Qiskit is an open-source SDK for working with quantum computers at the level of extended quantum circuits, operators, and primitives.

https://www.ibm.com/quantum/qiskit

Apache License 2.0

5.28k stars 2.37k forks source link

Update MCX synthesis to use `PhaseGate` instead of `U1Gate` #12977

Open Cryoris opened 3 months ago

Cryoris commented 3 months ago

Summary

We should update the synthesis methods to use the PhaseGate instead of the IBM-legacy U1Gate, if possible. See https://github.com/Qiskit/qiskit/pull/12961#discussion_r1719719621.

jakelishman commented 2 months ago

RZGate also should be fine if you prefer to centralise on the rx/ry/rz set to make the logic simpler - just needs a couple of local operations to account for the phase difference.