feat: Interpret byte operations#962
Open
nchappe wants to merge 8 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
VeIR Benchmarks
Details
| Benchmark suite | Current: 66806ab | Previous: 38815ba | Ratio |
|---|---|---|---|
add-fold-worklist/create |
2576500 ns (± 173970) |
1883000 ns (± 84571) |
1.37 |
add-fold-worklist/rewrite |
3878000 ns (± 55459) |
3549000 ns (± 89577) |
1.09 |
add-fold-worklist-local/create |
2549000 ns (± 83388) |
1817000 ns (± 53528) |
1.40 |
add-fold-worklist-local/rewrite |
3240000 ns (± 17601) |
2956000 ns (± 124977) |
1.10 |
add-zero-worklist/create |
2562000 ns (± 217083) |
1873000 ns (± 84086) |
1.37 |
add-zero-worklist/rewrite |
2521500 ns (± 94777) |
2262000 ns (± 82907) |
1.11 |
add-zero-reuse-worklist/create |
2179500 ns (± 216627) |
1579000 ns (± 48905) |
1.38 |
add-zero-reuse-worklist/rewrite |
2077500 ns (± 68566) |
1834000 ns (± 47410) |
1.13 |
mul-two-worklist/create |
2576500 ns (± 248359) |
1817000 ns (± 44072) |
1.42 |
mul-two-worklist/rewrite |
5459500 ns (± 141936) |
5025000 ns (± 64767) |
1.09 |
add-fold-forwards/create |
2506000 ns (± 233399) |
1854000 ns (± 89063) |
1.35 |
add-fold-forwards/rewrite |
2941000 ns (± 59903) |
2668000 ns (± 54288) |
1.10 |
add-zero-forwards/create |
2660500 ns (± 201249) |
1872000 ns (± 38079) |
1.42 |
add-zero-forwards/rewrite |
1936500 ns (± 64734) |
1732000 ns (± 26197) |
1.12 |
add-zero-reuse-forwards/create |
2315500 ns (± 153087) |
1549000 ns (± 57405) |
1.49 |
add-zero-reuse-forwards/rewrite |
1565000 ns (± 48272) |
1374000 ns (± 50677) |
1.14 |
mul-two-forwards/create |
2488000 ns (± 120447) |
1868500 ns (± 100200) |
1.33 |
mul-two-forwards/rewrite |
3603500 ns (± 53601) |
3297500 ns (± 89753) |
1.09 |
add-zero-reuse-first/create |
2289000 ns (± 197917) |
1549500 ns (± 58804) |
1.48 |
add-zero-reuse-first/rewrite |
9000 ns (± 2865) |
10000 ns (± 3126) |
0.90 |
add-zero-lots-of-reuse-first/create |
2299000 ns (± 187580) |
1553000 ns (± 19409) |
1.48 |
add-zero-lots-of-reuse-first/rewrite |
820500 ns (± 57283) |
754000 ns (± 13856) |
1.09 |
This comment was automatically generated by workflow using github-action-benchmark.
aa5d410 to
01599bf
Compare
There is one sorry for now because memory and bytes make different assumptions about poison.
464f0e0 to
66806ab
Compare
Contributor
Author
|
Fixed the sorry and added a few missing operations |
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.
shl, lshr, trunc, freeze, bitcast