Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

perf(es/codegen): Remove needless allocations #9978

Merged
merged 11 commits into from
Feb 5, 2025
Merged

Conversation

kdy1
Copy link
Member

@kdy1 kdy1 commented Jan 31, 2025

Description:

We don't need to allocate from get_quoted_utf16 in most cases.

@kdy1 kdy1 added this to the Planned milestone Jan 31, 2025
Copy link

changeset-bot bot commented Jan 31, 2025

🦋 Changeset detected

Latest commit: 09c1425

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

codspeed-hq bot commented Jan 31, 2025

CodSpeed Performance Report

Merging #9978 will degrade performances by 4.28%

Comparing kdy1:cg-alloc (09c1425) with main (963b088)

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 194 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
es/full/minify/libraries/lodash 230.6 ms 241 ms -4.28%
es/minifier/libs/terser 476.7 ms 462.4 ms +3.08%

@kdy1 kdy1 self-assigned this Feb 3, 2025
@kdy1 kdy1 marked this pull request as ready for review February 5, 2025 00:55
@kdy1 kdy1 requested review from a team as code owners February 5, 2025 00:55
kodiakhq[bot]
kodiakhq bot previously approved these changes Feb 5, 2025
kodiakhq[bot]
kodiakhq bot previously approved these changes Feb 5, 2025
@kdy1 kdy1 changed the title perf(es/codegen): Remove neelees allocations perf(es/codegen): Remove needless allocations Feb 5, 2025
@kdy1 kdy1 merged commit 9c89d57 into swc-project:main Feb 5, 2025
172 checks passed
@kdy1 kdy1 deleted the cg-alloc branch February 5, 2025 03:02
mischnic added a commit to vercel/next.js that referenced this pull request Feb 5, 2025
### What?

ChangeLog:
swc-project/swc@swc_core@v12.0.0...swc_core@v13.0.4

### Why?


 - To keep in sync
 - To drop `rustc-hash` v1
 - To apply swc-project/swc#9978

---------

Co-authored-by: Niklas Mischkulnig <4586894+mischnic@users.noreply.github.com>
@kdy1 kdy1 removed this from the Planned milestone Feb 8, 2025
@kdy1 kdy1 added this to the v1.10.15 milestone Feb 8, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant