-
Notifications
You must be signed in to change notification settings - Fork 40
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
Re-enable micro_speech test #176
Comments
This issue has 1 outstanding TODOs:
This comment was autogenerated by todo-backlinks |
j2kun
added a commit
to j2kun/heir
that referenced
this issue
May 23, 2024
At this point I think it's fair to say that the old heir-tosa-to-arith pass is not the way we want to move forward. Rather, we will opporuntistically fuse some of the affine loops and optimize the bodies jointly with yosys, instead of always fully unrolling. So this changes the heir-tosa-to-arith pipeline to simply be the prefix of the tosa-to-boolean-tfhe pipeline, without adding the secret related passes. Fixes google#176
j2kun
added a commit
to j2kun/heir
that referenced
this issue
May 23, 2024
At this point I think it's fair to say that the old heir-tosa-to-arith pass is not the way we want to move forward. Rather, we will opporuntistically fuse some of the affine loops and optimize the bodies jointly with yosys, instead of always fully unrolling. So this changes the heir-tosa-to-arith pipeline to simply be the prefix of the tosa-to-boolean-tfhe pipeline, without adding the secret related passes. Fixes google#176
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Re-enable the tests of the ML workflow once HEIR is in a place to execute it without it taking ~hours to run.
The text was updated successfully, but these errors were encountered: