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

LLVM unit tests #324

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

JoelleJS
Copy link
Contributor

@JoelleJS JoelleJS commented Jan 15, 2025

@JoelleJS JoelleJS changed the title LLVM unit tests (issue #312) LLVM unit tests Jan 15, 2025
@vosen
Copy link
Owner

vosen commented Jan 20, 2025

I don't see the Rust code to generate those tests. Not sure if I botched the explanation, your code is still in progress or you forgot to check it in or, but the tests are one part. The other part is Rust code: for each file there should be a rust unit test (generated by extending the existing macro that generates unit tests) to verify that .ll match LLVM bitcode generated by the compiler

@JoelleJS
Copy link
Contributor Author

@vosen Yep, still working on that part. Just wanted the .ll files committed

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