-
Notifications
You must be signed in to change notification settings - Fork 6
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
Bsg mul add unsigned dev #454
Conversation
…sys-log-filepath flag
…d main.rkt in order to separate it from the other PR
…d into bsg_mul_add_unsigned_dev
Changing solver back to Bitwuzla
testing cvc5 solver - stp didnt work so far - bitwuzla didnt work so far
adjusted params
testing yices
bin/main.rkt
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary change, please remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add this to your private .git/info/exclude
, not here!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you don't explicitly exclude this file, lit
will try to run it like a test, which is part of why you're getting errors in CI:
...
Unresolved Tests (3):
Lakeroad tests :: bsg_defines.sv
Lakeroad tests :: bsg_dff.sv
Lakeroad tests :: bsg_dff_chain.sv
...
Can you do two things:
- Put the files into a folder called
bsg_mul_add_unsigned
- Exclude the files so that
lit
doesn't try to run them
You can see an example here:
https://github.com/uwsampl/lakeroad/pull/447/files#
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please delete this whole folder, I think it was checked in accidentally!
lakeroad-private
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please get rid of this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GREAT WORK!!!!!!!!!!!!!
Thanks for being so persistent on this! Glad it's finally merged!
New (separate) PR to add the BaseJump STL mul-add-unsigned file into Lakeroad