cfallin opened PR #4220 from cranelift-readme
to main
:
Our README was starting to show its age; it did not reflect the current
status of Cranelift well with respect to production maturity, current
supported backends, or performance. This PR makes a pass over the
"Status" section to fix that. It also removes some old/out-of-date
details, likeno_std
support (which has bitrotted).I got the quoted performance numbers by capturing a zoomed-in screenshot
of the bar plot in the linked paper and measuring their lengths in pixels (!).
I wish more research made raw numbers more easily available! Ideally we'd
source performance claims from Sightglass continuous benchmarking,
once that exists.<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
cfallin requested alexcrichton for a review on PR #4220.
cfallin requested abrown for a review on PR #4220.
cfallin requested fitzgen for a review on PR #4220.
cfallin requested akirilov-arm for a review on PR #4220.
fitzgen submitted PR review.
fitzgen created PR review comment:
#1032
won't automatically become a link here, so it needs to be explicitly linked.
fitzgen submitted PR review.
cfallin updated PR #4220 from cranelift-readme
to main
.
cfallin created PR review comment:
Fixed, thanks!
cfallin submitted PR review.
cfallin merged PR #4220.
Last updated: Nov 22 2024 at 16:03 UTC