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

WIP: Integration tests #121

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

Conversation

boomanaiden154
Copy link
Collaborator

This PR adds in integration tests to ensure that (most) of the current training workflows work as expected. This patch adds in one workflow based on the current demo that evaluates the workflow for the inlining case using fuschia as a corpus (not currently working, need to do some more debugging). For the regalloc case, I'm currently evaluating it against a chromium corpus with the bitcode embedding patch added in #117. The only major workflows that I think I'm currently missing would be corpora with ThinLTO, particularly inlining for chromium since that got upstreamed recently, but I'm planning on working on that in a separate patch if so desired.

TODO:

  • Get Fuchsia inlining demo fully functional. Fuchsia build currently fails due to a missing .code_patches segment when the build system tries to run llvm-objcopy on something.
  • Rebase against main once Add chromium patch #117 gets merged.
  • Move everything over to TFLite. Probably will be a separate patch to make the docker setup similar to what the buildbots are doing currently.

@boomanaiden154 boomanaiden154 mentioned this pull request Sep 22, 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.

1 participant