| Commit message (Collapse) | Author | Age |
| |
|
| |
|
|
|
|
|
|
| |
This required tending to montladder not being proved Feq-preserving
(sidestepped by proving for all P = 0), and then some wrestling with
scalarmult to show the right-hand side was indeed zero when x is (0, 0).
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
This is a one-line change in generate_parameters.py (plus some
whitespace trimming), and running `make regenerate-curves`
This handles part of #294
|
|
|
|
|
|
|
|
|
| |
After | File Name | Before || Change | % Change
-------------------------------------------------------------------------
1m11.42s | Total | 1m53.75s || -0m42.33s | -37.21%
-------------------------------------------------------------------------
1m06.02s | Curves/Weierstrass/Jacobian | 1m53.76s || -0m47.73s | -41.96%
0m05.40s | Util/FsatzAutoLemmas | N/A || +0m05.40s | ∞
|
|
|
|
| |
We no longer seem to need the stronger hypothesis.
|
|
|
|
|
|
|
|
| |
After | File Name | Before || Change | % Change
---------------------------------------------------------------------------
1m52.31s | Total | 13m36.75s || -11m44.44s | -86.24%
---------------------------------------------------------------------------
1m52.31s | Curves/Weierstrass/Jacobian | 13m36.75s || -11m44.44s | -86.24%
|
|
|
|
|
|
| |
This reverts commit d33d8be154dbce048ac10d82bc0b39468abd5fdb.
Hmm, apparently there's an error on Qed... maybe a bug in fsatz?
|
|
|
|
| |
We no longer seem to need the stronger hypothesis.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Also refresh things after PR#289. This too just updated the comments, as
expected.
|
| |
|
| |
|
|
|
|
|
| |
Refresh things after PR#289, though that just updated the comments, as
expected.
|
|
|
|
|
|
|
| |
This notation system is fragile and kludgy.
This discovered from @davidben's
https://github.com/mit-plv/fiat-crypto/pull/289/commits/ff0fb38346dde67abef982d6305595216d18519b#r159793723
|
|
|
|
|
| |
We do this by adding notations for addcarryx and subborrow for all of
the smaller-than-max-bitwidth sizes of arguments.
|
|
|
|
| |
This handles bullet 3 of #288
|
|
|
|
| |
This handles bullet point 1 of #288
|
|
|
|
| |
This handles bullet 2 of #288
|
|
|
|
| |
This handles bullet 2 of #288
|
|
|
|
|
|
|
|
|
|
|
| |
This closes #286
This is actually a +1,-1 diff in the python script generating the
notations, plus running it and rebuilding:
```diff
-ADD_CARRY_SUB_BORROW_SIZES = (32, 64, 128, 51)
+ADD_CARRY_SUB_BORROW_SIZES = (32, 64, 128, 25, 26, 51)
```
|
|
|
|
|
|
| |
This makes it easier to add support for more kinds of addcarryx, etc,
and also add `: expr_scope` to work around changes from
https://github.com/coq/coq/pull/873
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
* clean up src/Experiments/Loops.v, add While and For
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* [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...
* [demo] Add more namespacing
* Update llet notation, update is_known_const name
As per code review suggestions
* Namespace var_context, add some coqbug references
* Rename is_type_arg to is_template_parameter
As per https://github.com/mit-plv/fiat-crypto/pull/275#discussion_r153036059
* Simplify the logic around delayed arguments a bit
We no longer pass around dummy markers in the tuple of arguments.
* [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
https://github.com/coq/coq/issues/6252#issuecomment-347041995
* [demo] respond to code review, add comments
* Update documentation with suggestions from Andres
|
|
|
|
| |
As per https://github.com/mit-plv/fiat-crypto/pull/275#discussion_r153084144
|
|
|
|
| |
This sets the level and associativity in a common place, and will be useful for PHOAS application nodes
|
| |
|
| |
|
| |
|
|
|
|
| |
This will make it easier to reify
|
|
|
|
|
| |
As per
https://github.com/mit-plv/fiat-crypto/pull/271/commits/76634efa66aa9085a1e410c04f930dd5645126df#r152719891
|