Skip to content

Comments

perf(transformer): React JSX reuse same Atoms#3521

Merged
graphite-app[bot] merged 1 commit intomainfrom
06-03-perf_transformer_react_jsx_reuse_same_atom_s
Jun 4, 2024
Merged

perf(transformer): React JSX reuse same Atoms#3521
graphite-app[bot] merged 1 commit intomainfrom
06-03-perf_transformer_react_jsx_reuse_same_atom_s

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Jun 3, 2024

Previously new Atoms were generated and allocated into arena each time e.g. _React is used. This PR changes that to allocate each string into arena once only, and then re-use that same Atom over and over.

@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 3, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

Copy link
Member Author

overlookmotel commented Jun 3, 2024

@github-actions github-actions bot added the A-transformer Area - Transformer / Transpiler label Jun 3, 2024
@overlookmotel overlookmotel marked this pull request as ready for review June 3, 2024 16:30
@codspeed-hq
Copy link

codspeed-hq bot commented Jun 3, 2024

CodSpeed Performance Report

Merging #3521 will not alter performance

Comparing 06-03-perf_transformer_react_jsx_reuse_same_atom_s (f3a755c) with main (837776e)

Summary

✅ 22 untouched benchmarks

@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 4, 2024

Merge activity

Previously new `Atom`s were generated and allocated into arena each time e.g. `_React` is used. This PR changes that to allocate each string into arena once only, and then re-use that same `Atom` over and over.
@Boshen Boshen force-pushed the 06-03-refactor_traverse_generate_uid_return_symbolid_ branch from 5e5b1a0 to 7bbd3da Compare June 4, 2024 03:02
@Boshen Boshen force-pushed the 06-03-perf_transformer_react_jsx_reuse_same_atom_s branch from 8f0d568 to f3a755c Compare June 4, 2024 03:03
@Boshen Boshen changed the base branch from 06-03-refactor_traverse_generate_uid_return_symbolid_ to main June 4, 2024 03:06
@graphite-app graphite-app bot merged commit f3a755c into main Jun 4, 2024
@graphite-app graphite-app bot deleted the 06-03-perf_transformer_react_jsx_reuse_same_atom_s branch June 4, 2024 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-transformer Area - Transformer / Transpiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant