-
Notifications
You must be signed in to change notification settings - Fork 159
Speedup EQF_INTRO #1658
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
base: develop
Are you sure you want to change the base?
Speedup EQF_INTRO #1658
Conversation
Here are some numbers for my machine
|
Curious that the one with |
There's a bunch of indirection of creating a binarymap is my guess |
6b65ec4
to
a7920b4
Compare
I've realized that IMP_ANTISYM_RULE is really bad because the first SPEC could be a really big term and now the second SPEC needs to traverse that whole term |
SYM_CONV also faces the same problem |
I have no idea how to fix this |
Or more like I don't know the proper way to fix this. I can make opentheory kernel use the old version of EQF_INTRO but I have no idea whats causing the failure |
Is this th60 failing or th61 failing?
|
HOL/src/boss/prove_base_assumsScript.sml Lines 1088 to 1104 in e0253e9
|
HOL/src/boss/prove_base_assumsScript.sml Lines 1099 to 1115 in e0253e9
or th62 |
Ok goals is generated and can change and |
1290dbb
to
81edc43
Compare
Here is some testing scripts