perf(transformer/arrow-function): create super method binding names lazily

[overlookmotel]

Key super getter/setter hash map with original property name &str, instead of generated binding name. This allows a couple of improvements:

• Generate binding names for super getters/setters lazily, only when they're created, rather than every time you encounter a super.
• Don't allocate strings into arena which are never used as part of AST. Use CompactString for building those strings instead.

[graphite-app]

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

Add the label “0-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.

[overlookmotel]

• perf(transformer/arrow-function): reduce string comparisons #7318
• test(transformer/async-to-generator): failing test for nested supers #7314
• perf(transformer/arrow-functions): move arguments transform checks to aid inlining #7322
• perf(transformer/arrow-functions): store state of whether arguments needs transform #7321
• refactor(transformer/arrow-function): comments on possible improvement #7320
• refactor(transformer/arrow-function): reserve correct capacity for Vec #7319
• perf(transformer/arrow-function): create super method binding names lazily #7313 👈 (View in Graphite)
• refactor(transformer/arrow-functions): use IndexMap for super getter/setters #7317
• perf(transformer/arrow-function): optimize generate_super_binding_name #7312
• fix(transformer/arrow-function): handle unicode when capitalizing property name #7311
• refactor(transformer/arrow-function): generate_super_binding_name take &str and &TraverseCtx #7310
• fix(transformer/async-to-generator): arguments isn't correct after transformation #7234
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Performance Report

Merging #7313 will not alter performance

_{Comparing 11-16-perf_transformer_arrow-function_super_method_binding_names_lazily (e09d2df) with 11-16-refactor_transformer_arrow-functions_use_indexmap_for_super_getter_setters (5d85386)}

Summary

✅ 30 untouched benchmarks

[Dunqing]

Merge activity

• Nov 17, 12:03 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Nov 17, 12:03 AM EST: A user added this pull request to the Graphite merge queue.
• Nov 17, 12:15 AM EST: The Graphite merge queue removed this pull request due to downstack failures on PR #7310.
• Nov 17, 12:16 AM EST: The Graphite merge queue removed this pull request due to downstack failures on PR #7310.
• Nov 17, 5:31 AM EST: A user added this pull request to the Graphite merge queue.
• Nov 17, 5:42 AM EST: A user merged this pull request with the Graphite merge queue.