Qiskit / qiskit / 11460709254

89%

main: 88%

Build Type

push

github

Committed by web-flow

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.

Run Details

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

11 existing lines in 4 files now uncovered.

74577 of 84135 relevant lines covered (88.64%)

364195.81 hits per line