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

Suggestion: clear unused assumptions first #118

Open
QinshiWang opened this issue Dec 26, 2021 · 2 comments
Open

Suggestion: clear unused assumptions first #118

QinshiWang opened this issue Dec 26, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@QinshiWang
Copy link

It will be helpful if the tactic generated by hammer can first remove unused assumptions. In my simple example, (a) takes 0.046s while (b) takes 5.888s.

(a)
clear -H13.
Time hauto use: Z.add_0_r unfold: sval_refine.
(b)
Time hauto use: Z.add_0_r unfold: sval_refine.

It's worth mentioning that I added clear -H13 at the beginning because otherwise hammer would time out. But that doesn't matter for the main point.

@QinshiWang QinshiWang changed the title Clear unused assumptions first Suggestion: clear unused assumptions first Dec 26, 2021
@lukaszcz lukaszcz added the enhancement New feature or request label Dec 26, 2021
@lukaszcz
Copy link
Owner

This shouldn't be too hard. I'll do it sometime in the future, in several months perhaps.

@QinshiWang
Copy link
Author

That'll be great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants