-
Notifications
You must be signed in to change notification settings - Fork 147
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
[demo] Add reification in src/Experiments/SimplyTypedArithmetic.v #275
Merged
Commits on Nov 26, 2017
-
[demo] Add reification in src/Experiments/SimplyTypedArithmetic.v
It's rather verbose, unfortunately. The reification also doesn't have any of the nice debugging features of the version of reification in Compilers, because that's even more boilerplate. Not sure if I should add that back in, at the moment. Also, for some strange reason, places where `constr`s fail to typecheck seem to induce backtracking where I don't think they should, and I'm not sure what's going on...
Configuration menu - View commit details
-
Copy full SHA for 53b5487 - Browse repository at this point
Copy the full SHA 53b5487View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8a51550 - Browse repository at this point
Copy the full SHA 8a51550View commit details -
Update llet notation, update is_known_const name
As per code review suggestions
Configuration menu - View commit details
-
Copy full SHA for 055b3c2 - Browse repository at this point
Copy the full SHA 055b3c2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 547c195 - Browse repository at this point
Copy the full SHA 547c195View commit details -
Configuration menu - View commit details
-
Copy full SHA for fd27382 - Browse repository at this point
Copy the full SHA fd27382View commit details -
Simplify the logic around delayed arguments a bit
We no longer pass around dummy markers in the tuple of arguments.
Configuration menu - View commit details
-
Copy full SHA for a11ceeb - Browse repository at this point
Copy the full SHA a11ceebView commit details -
[demo] More informative reification error messages
This time without exponential slowdown in failure cases and without needing to manually think up all of the possible errors and write them out. Possible thanks to Hugo's comment at coq/coq#6252 (comment)
Configuration menu - View commit details
-
Copy full SHA for 3bf1476 - Browse repository at this point
Copy the full SHA 3bf1476View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7a6a548 - Browse repository at this point
Copy the full SHA 7a6a548View commit details -
Configuration menu - View commit details
-
Copy full SHA for ba9f695 - Browse repository at this point
Copy the full SHA ba9f695View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.