lukaszcz / coqhammer

CoqHammer: An Automated Reasoning Hammer Tool for Coq - Proof Automation for Dependent Type Theory
Other
217 stars 31 forks source link

Suggestion: clear unused assumptions first #118

Open QinshiWang opened 2 years ago

QinshiWang commented 2 years ago

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.

lukaszcz commented 2 years ago

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

QinshiWang commented 2 years ago

That'll be great!