Qiskit / qiskit / 15279236676

88%

Build Type

push

github

Committed by web-flow

Commit Message

Pre-allocate bits storage in CircuitData::new (#14463)

* Pre-allocate bits storage in CircuitData::new

When calling `CircuitData::new()` if the qubits or clbits arguments are
passed in as Some we know how large the allocation for the
ObjectRegistry and BitLocations objects are going to be. Right now we
don't preallocate this space and rely on the internal `HashMap`s and
`Vec`s to grow as we populate the mappings. This ends up being wasted
time as we reallocate the underlying storage. For very wide circuits
with a large number of qubits or clbits the extra allocation time can
be quite substantial relative to the other operations `new()` performs.
This commit fixes this issue by calling the `with_capacity()` methods if
qubits or clbits is set to preallocate the ObjectRegistry and
BitLocations.

* Simplify code

Run Details

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

6 existing lines in 4 files now uncovered.

78476 of 88846 relevant lines covered (88.33%)

466628.59 hits per line