-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Fix illegal instruction error on arm64 by enabling no_opt_arch
feature for mimalloc-safe
#6055
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
Fix illegal instruction error on arm64 by enabling no_opt_arch
feature for mimalloc-safe
#6055
Conversation
…ure for mimalloc-safe
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6055 +/- ##
=======================================
Coverage 98.79% 98.79%
=======================================
Files 271 271
Lines 10601 10601
Branches 2830 2830
=======================================
Hits 10473 10473
Misses 88 88
Partials 40 40 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Thanks! Unfortunately I do not have access to my computer for the next two weeks. If you want an earlier release, we will need to see if I can give @TrickyPi publishing rights. |
That would be great 🙏 |
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.
Let's try this
This PR has been released as part of [email protected]. You can test it via |
initially introduced by rollup#6055 for linux, this did not include android devices as it is considerd its own operating system with linux being the "vendor"; triplets are quite inconsistent and confusing at times! this change conservatively disables arch-specific optimizations for all aarch64 targets, to ensure broad compatibility. closes rollup#6047
* build(rust): mimalloc-safe/no_opt_arch on aarch64 initially introduced by #6055 for linux, this did not include android devices as it is considerd its own operating system with linux being the "vendor"; triplets are quite inconsistent and confusing at times! this change conservatively disables arch-specific optimizations for all aarch64 targets, to ensure broad compatibility. closes #6047 * build(rust): exclude Apple from arch-specific opt-out
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
Description
I don't have a aarch64 device so I cannot verify if this fix works. I found this fix in oxc-project/oxc#11045.
The cause is probably the same https://github.com/rollup/rollup/pull/6027/files#diff-232adeab3538fc2304f20f64aee234e2b54a5b830b5de1464ab403fb5e095228L13-R22, #6047 (comment).