Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Add QubitUnitaryFusion pass from development tutorial #880

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Qottmann
Copy link

@Qottmann Qottmann commented Jun 27, 2024

Trying to add the QubitUnitaryFusion pass from the compiler passes development site. Struggling with the very basics and boilerplate 馃珷

My approach: mimick behavior from ChainedHadamard.cpp and remove_chained_self_inverse.cpp, take on advice added in #872 and use code snippets from compiler passes. The error messages are rather cryptic at this point so not sure how to continue.

namespace catalyst {
namespace quantum {

#define GEN_PASS_DEF_QubitUnitaryFusionPASS
Copy link
Contributor

Choose a reason for hiding this comment

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

#define GEN_PASS_DEF_QUBITUNITARYFUSIONPASS

Copy link

github-actions bot commented Jun 27, 2024

Hello. You may have forgotten to update the changelog!
Please edit doc/changelog.md on your branch with:
A one-to-two sentence description of the change. You may include a small working example for new features.
A link back to this PR.
Your name (or GitHub username) in the contributors section.

Paul: No need for changelog since this is do-not-merge


#define DEBUG_TYPE "qubit-unitary-fusion"

#include "mlir/Dialect/Linalg/Ops.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this file does not exist. The closest I can find is #include "mlir/Dialect/Linalg/IR/Linalg.h", so I would go find in the mlir repo where the operations you want are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants