perf(mangler): use combined exclusion set for name generation#17381
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
614513b to
4044df2
Compare
3d03a51 to
e0032ba
Compare
Build a single FxHashSet containing all excluded names (unresolved references, keep_name_names, and conditionally root_bindings) instead of doing 3-5 separate hash lookups per candidate name in the generation loop.
e0032ba to
fbd84ef
Compare
CodSpeed Performance ReportMerging #17381 will degrade performance by 15.1%Comparing Summary
Benchmarks breakdown
Footnotes
|

Build a single FxHashSet containing all excluded names (unresolved references, keep_name_names, and conditionally root_bindings) instead of doing 3-5 separate hash lookups per candidate name in the generation loop.