-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat(rolldown): introduce rolldown_rayon #846
feat(rolldown): introduce rolldown_rayon #846
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @Brooooooklyn and the rest of your teammates on Graphite |
✅ Deploy Preview for rolldown-rs canceled.
|
8143045
to
6fdbf8f
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #846 +/- ##
==========================================
- Coverage 80.65% 80.52% -0.14%
==========================================
Files 133 133
Lines 6731 6685 -46
==========================================
- Hits 5429 5383 -46
Misses 1302 1302 ☔ View full report in Codecov by Sentry. |
6fdbf8f
to
f0fd95d
Compare
hhh. I was writing macros about this. I guess they are the same. |
macro can always produce better performance because it outputs the optimized codes in the compile stage. |
CodSpeed Performance ReportMerging #846 will not alter performanceComparing Summary
|
Let's merge it first. I want to do some changes after mering.
Something like swc did in ($name:ident.into_iter().$operator:ident($($rest:expr)*), $threshold:expr) => {
if $name.len() >= $threshold {
use rayon::prelude::*;
$name.into_par_iter().$operator($($rest)*)
} else {
$name.into_iter().$operator($($rest)*)
}
}; |
Reminds of |
Description
Introduce
rolldown_rayon
crate to make it easier to write rayon code on the non-rayon supported platforms.