[WIP] dependently typed implementation of peephole rewrite verification#492
Draft
math-fehr wants to merge 3 commits into
Draft
[WIP] dependently typed implementation of peephole rewrite verification#492math-fehr wants to merge 3 commits into
math-fehr wants to merge 3 commits into
Conversation
Contributor
There was a problem hiding this comment.
VeIR Benchmarks
Details
| Benchmark suite | Current: 862a0e8 | Previous: bb6ce4c | Ratio |
|---|---|---|---|
add-fold-worklist/create |
1893000 ns |
2235000 ns |
0.85 |
add-fold-worklist/rewrite |
3447000 ns |
3738000 ns |
0.92 |
add-fold-worklist-local/create |
1848000 ns |
2196000 ns |
0.84 |
add-fold-worklist-local/rewrite |
2857000 ns |
3048000 ns |
0.94 |
add-zero-worklist/create |
1860000 ns |
2247000 ns |
0.83 |
add-zero-worklist/rewrite |
2199000 ns |
2418000 ns |
0.91 |
add-zero-reuse-worklist/create |
1546000 ns |
1941000 ns |
0.80 |
add-zero-reuse-worklist/rewrite |
1761000 ns |
2009000 ns |
0.88 |
mul-two-worklist/create |
1848000 ns |
2293000 ns |
0.81 |
mul-two-worklist/rewrite |
4810000 ns |
5320000 ns |
0.90 |
add-fold-forwards/create |
1830000 ns |
2326000 ns |
0.79 |
add-fold-forwards/rewrite |
2750000 ns |
3023000 ns |
0.91 |
add-zero-forwards/create |
1857000 ns |
2258000 ns |
0.82 |
add-zero-forwards/rewrite |
1781000 ns |
1919000 ns |
0.93 |
add-zero-reuse-forwards/create |
1537000 ns |
1884000 ns |
0.82 |
add-zero-reuse-forwards/rewrite |
1387000 ns |
1541000 ns |
0.90 |
mul-two-forwards/create |
1904000 ns |
2192000 ns |
0.87 |
mul-two-forwards/rewrite |
3233000 ns |
3681000 ns |
0.88 |
add-zero-reuse-first/create |
1562000 ns |
1867000 ns |
0.84 |
add-zero-reuse-first/rewrite |
9000 ns |
8000 ns |
1.13 |
add-zero-lots-of-reuse-first/create |
1572000 ns |
1832000 ns |
0.86 |
add-zero-lots-of-reuse-first/rewrite |
759000 ns |
767000 ns |
0.99 |
This comment was automatically generated by workflow using github-action-benchmark.
2fcb5b9 to
f4dc8ad
Compare
f4dc8ad to
e8dde38
Compare
bf52e3c to
4a5f847
Compare
4a5f847 to
862a0e8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Exploration of using dependent types in the interpreter to make proofs much simpler in the peephole rewrite verification