Qiskit / qiskit / 26457875953

88%

Build Type

push

github

Committed by web-flow

Commit Message

Add rust short circuit to unitariy synthesis if no nodes to synthesize (#16274)

The unitary synthesis pass is a no-op if there are no gates to
synthesize in the circuit. The dag maintains a count of all the
operations it contains so it is trivial to check if there are any
operations to synthesize up front before running the pass. This check
existed in the python code for the Python version of the pass. But it
was missing from the rust code which means when the pass is run from C,
either as a standalone function or as part of the C transpiler, we were
not doing this short circuit. This adds the skip logic ot the Rust code
for the pass so that C can benefit from the performance benefit of not
iterating over and reconstructing the dag if it's not necessary. The
python short circuit is left in place because the python pass may not
call the rust code (if a unitary synthesis plugin is called).

Coverage Stats

37 of 41 new or added lines in 4 files covered. (90.24%)

9 existing lines in 4 files now uncovered.

108502 of 123980 relevant lines covered (87.52%)

954965.52 hits per line