-
Notifications
You must be signed in to change notification settings - Fork 234
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
[Merged by Bors] - feat: add Submodule.Quotient.instBoundedSMul
#10520
Conversation
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.
LGTM!
Should it be !benched? I always wonder about possible performance issues of instances...
I don't think that performance should be an issue here. |
🚀 Pull request has been placed on the maintainer queue by mcdoll. |
Submodule.instBoundedSMul
Submodule.Quotient.instBoundedSMul
Thanks! 🎉 |
Pull request successfully merged into master. Build succeeded: |
Submodule.Quotient.instBoundedSMul
Submodule.Quotient.instBoundedSMul
The proof needs minimal modifications to go through for the inequality case.