Skip to content

Conversation

@stormslowly
Copy link
Contributor

To check if tracing has a performance cost

Description:

BREAKING CHANGE:

Related issue (if exists):

@changeset-bot
Copy link

changeset-bot bot commented Nov 6, 2025

⚠️ No Changeset found

Latest commit: 182dc3a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

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

@kdy1
Copy link
Member

kdy1 commented Nov 6, 2025

It has a noticeable cost. You can try enabling tracing/max_level_off

@stormslowly stormslowly force-pushed the chore/try_no_instrument branch from f5721a4 to ecd55a6 Compare November 6, 2025 03:34
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 6, 2025

CodSpeed Performance Report

Merging #11237 will not alter performance

Comparing stormslowly:chore/try_no_instrument (182dc3a) with main (d93ec90)

Summary

✅ 138 untouched

@stormslowly
Copy link
Contributor Author

In rspack-resolver, when I enable max_level_off and release_max_level_off, I still get 2~15% performance loss in benchmark.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Binary Sizes

File Size
swc.linux-x64-gnu.node 31M (31818120 bytes)

Commit: f4fda6e

@kdy1
Copy link
Member

kdy1 commented Nov 6, 2025

I think it's a good idea to remove all tracing code, considering the perf difference

@stormslowly
Copy link
Contributor Author

According to codspeed's report, the most improved step is enabling max_level_off.
That's quite different from rspack-resolver's case.
I will do some digging to find the season and figure out a way to improve performance and keep trackability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants