-
Notifications
You must be signed in to change notification settings - Fork 111
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
update toolchain and revert llvm issue patch #1004
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1004 +/- ##
==========================================
- Coverage 92.92% 92.92% -0.01%
==========================================
Files 105 105
Lines 14243 14242 -1
Branches 14243 14242 -1
==========================================
- Hits 13236 13235 -1
Misses 927 927
Partials 80 80 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2
.
Benchmark suite | Current: ec15a53 | Previous: 394f839 | Ratio |
---|---|---|---|
iffts/simd ifft/22 |
12692101 ns/iter (± 113987 ) |
6272224 ns/iter (± 205833 ) |
2.02 |
merkle throughput/simd merkle |
30917673 ns/iter (± 650900 ) |
13123349 ns/iter (± 484950 ) |
2.36 |
This comment was automatically generated by workflow using github-action-benchmark.
CC: @shaharsamocha7
056648f
to
572d70a
Compare
572d70a
to
ec15a53
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 15 files reviewed, 1 unresolved discussion (waiting on @ohad-starkware)
a discussion (no related file):
They fixed your issue?
No description provided.