Qiskit / qiskit / 17407774945

88%

Build Type

push

github

Committed by web-flow

Commit Message

C API: Burn `ConsolidateBlocks` (#14751)

* Add: C FFI for `ConsolidateBlocks`
- Add `qk_transpiler_pass_standalone_consolidate_blocks` to expose a version of the `ConsolidateBlocks` pass to C. The pass runs on an instance of `CircuitData` and an optional `Target`.

* Chore: Add tests and fix formatting.

* Fix: C test deallocates

* Fix: Add missing void in function signature for c tests,

* Fix: Free circuit instruction instance afterwards.

* FIx: Add missing `qk_count_ops_free` to Ctests

* Fix: Modify the Circuit in place.

* Docs: Add release note

* Chore: Add test with target

* Fix: Incorrect formatting on c tests.

* Fix: Address review comments.
- Quit using malloc in favor of stack allocations.
- Remove dependency on synthesis module from `cext`.
- Add mention of `NaN` on docstring.

* Fix: Pivot to freeing the allocated space.

* Fix: Adapt to #14930

* Fix: Redo stack approach correctly

Run Details

25 of 26 new or added lines in 1 file covered. (96.15%)

13 existing lines in 3 files now uncovered.

91306 of 103322 relevant lines covered (88.37%)

480954.62 hits per line