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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new CodeGenModel trait for generation to allow extensibility #47

Closed
wants to merge 3 commits into from

Conversation

ss2165
Copy link

@ss2165 ss2165 commented Feb 8, 2022

Baased on #46, only the final few commits are new.

Other tools may wish to use the same code generation utilities but with a different "SemanticModel", this trait allows for that.

As there are accessors for all members SemanticModel members are made private.

Instruction code gen utilities are made public for reuse.

These changes are used here: to protoype QIR code generation from the tket JSON IR.

@ss2165 ss2165 changed the base branch from main to iadavis/base-profile February 8, 2022 17:36
@idavis idavis deleted the branch qir-alliance:main February 8, 2022 21:02
@idavis idavis closed this Feb 8, 2022
@idavis
Copy link
Collaborator

idavis commented Feb 8, 2022

This was accidentally closed by GH when the PR was merged.

@idavis idavis reopened this Feb 8, 2022
@ss2165 ss2165 changed the base branch from iadavis/base-profile to main February 9, 2022 09:07
allows extensibility to other structures that implement
CodeGenModel that can be written out to QIR
@ghost
Copy link

ghost commented Mar 7, 2022

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

❌ ss2165 sign now
You have signed the CLA already but the status is still pending? Let us recheck it.

@ss2165
Copy link
Author

ss2165 commented Apr 27, 2022

Closing as this is not a change critical to my development and may not align well with more recent changes to main.

@ss2165 ss2165 closed this Apr 27, 2022
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.

2 participants