Skip to content

Add patterns to lower qecp subroutines to quantum#2835

Merged
multiphaseCFD merged 79 commits into
mainfrom
shuli/lower_qecp_subroutines_to_quantum
May 19, 2026
Merged

Add patterns to lower qecp subroutines to quantum#2835
multiphaseCFD merged 79 commits into
mainfrom
shuli/lower_qecp_subroutines_to_quantum

Conversation

@multiphaseCFD
Copy link
Copy Markdown
Member

@multiphaseCFD multiphaseCFD commented May 13, 2026

Before submitting

Please complete the following checklist when submitting a PR:

  • All new functions and code must be clearly commented and documented.

  • Ensure that code is properly formatted by running make format.
    The latest version of black and clang-format-20 are used in CI/CD to check formatting.

  • All new features must include a unit test.
    Integration and frontend tests should be added to frontend/test,
    Quantum dialect and MLIR tests should be added to mlir/test, and
    Runtime tests should be added to runtime/tests.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.


Context:

[sc-119621]

In the qecp layer, several subroutines for qeccycle, noise, encoding, measure, tranversal_gates that accept and return qec.codeblock are created. This PR is to lowering the signature of those subroutines to quantum types as well as update the callOp operations in the IR

Description of the Change:

Benefits:

Possible Drawbacks:

Related GitHub Issues:

Copy link
Copy Markdown
Contributor

@lillian542 lillian542 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

Copy link
Copy Markdown
Contributor

@joeycarter joeycarter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @multiphaseCFD! Two comments to get started.

Comment thread frontend/catalyst/python_interface/transforms/qecp/convert_qecp_to_quantum.py Outdated
@multiphaseCFD multiphaseCFD requested a review from joeycarter May 19, 2026 19:14
Copy link
Copy Markdown
Contributor

@joeycarter joeycarter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks @multiphaseCFD! Just had one final suggestion.

Base automatically changed from shuli/qecp-to-quantum-gate-op-conversion to main May 19, 2026 21:10
@multiphaseCFD multiphaseCFD merged commit 680310a into main May 19, 2026
37 of 38 checks passed
@multiphaseCFD multiphaseCFD deleted the shuli/lower_qecp_subroutines_to_quantum branch May 19, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants