[WIP] C API: Burn ConsolidateBlocks
#14751
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
raynelfss
added
Changelog: New Feature
raynelfss
changed the title
ConsolidateBlocksConsolidateBlocks
raynelfss
changed the title
ConsolidateBlocksConsolidateBlocks
4 tasks
Pull Request Test Coverage Report for Build 16524455693Details
💛 - Coveralls |
mtreinish
reviewed
Jul 17, 2025
|
|
||
| /// Helper function that extracts the decomposer and basis gate directly from the [Target]. | ||
| #[inline] | ||
| fn get_decomposer_and_basis_gate( |
There was a problem hiding this comment.
This should be integrated into the internal rust API. We will need a rust entrypoint that has the signature of the C API to call from the transpile() function. The C API should be doing appreciable logic like this it should just mirror the request to the rust api and handle basic type conversion from the limitations of the FFI boundary to something more rust native.
raynelfss
force-pushed
the
burn-consolidate-blocks
branch
3 times, most recently
from
41f7c26 to
3fa83b9
Compare
Built on top of Qiskit#14748. The following commit adds the ability for the `ConsolidateBlocks` pass to instantiate decomposers and choose a basis gate based on the provided `Target`. In the case that a `Target` is not provided, it will default to a `TwoBasisDecomposer` with a `CX` gate.
- 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`.
raynelfss
force-pushed
the
burn-consolidate-blocks
branch
from
f526557 to
2e079bc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Add
qk_transpiler_pass_standalone_consolidate_blocksto expose a version of theConsolidateBlockspass to C. The pass runs on an instance ofCircuitDataand an optionalTarget.Pre-requisites:
ConsolidateBlocks#14748ConsolidateBlocks#14776Details and comments
Built on top of #14748 needs to be rebased after it merges.
Closes #14447