Qiskit / qiskit / 11461529274

90%

main: 89%

Build Type

Pull #13352

github

Committed by mergify[bot]

Commit Message

Fix OpenQASM 2 `gate` definitions following a shadowed built-in `gate` (#13340)

Previously, when given an OpenQASM 2 program such as

    OPENQASM 2.0;
    gate builtin a { U(0, 0, 0) a; }
    gate not_builtin a { U(pi, pi, pi) a; }
    qreg q[1];

    not_builtin q[0];

and a set of `custom_instructions` including a `builtin=True` definition
for `builtin` (but not for `not_builtin`), the Rust and Python sides
would get themselves out-of-sync and output a gate that matched a prior
definition, rather than `not_builtin`.

This was because the Rust side was still issuing `DefineGate` bytecode
instructions, even for gates whose OpenQASM 2 definitions should have
been ignored, so Python-space thought there were more gates than
Rust-space thought there were.

(cherry picked from commit 9a1d8d3ae)

Pull Request Pull Request #13352: Fix OpenQASM 2 `gate` definitions following a shadowed built-in `gate` (backport #13340)

Run Details

6 of 6 new or added lines in 1 file covered. (100.0%)

8 existing lines in 1 file now uncovered.

66886 of 74527 relevant lines covered (89.75%)

219815.11 hits per line