Skip to content

Pre-allocate mutation vectors in Differentiator to reduce reallocation overhead#55819

Open
sammy-SC wants to merge 1 commit intofacebook:mainfrom
sammy-SC:export-D94096006
Open

Pre-allocate mutation vectors in Differentiator to reduce reallocation overhead#55819
sammy-SC wants to merge 1 commit intofacebook:mainfrom
sammy-SC:export-D94096006

Conversation

@sammy-SC
Copy link
Contributor

Summary:
Changelog: [Internal]

speed up differentiator

Test Before (ms) After (ms) Improvement
100 uncollapsable views 3.0 2.7 +10%
1000 uncollapsable views 91.1 50.5 +45%
100 views w/ many props 9.8 9.2 +6%
1000 views w/ many props 147.4 116.7 +21%
1500 views w/ many props 143.1 136.7 +5%

Differential Revision: D94096006

…n overhead

Summary:
Changelog: [Internal]

speed up differentiator

| Test                          | Before (ms) | After (ms) | Improvement |
|-------------------------------|-------------|------------|-------------|
| 100 uncollapsable views       | 3.0         | 2.7        | +10%        |
| 1000 uncollapsable views      | 91.1        | 50.5       | +45%        |
| 100 views w/ many props       | 9.8         | 9.2        | +6%         |
| 1000 views w/ many props      | 147.4       | 116.7      | +21%        |
| 1500 views w/ many props      | 143.1       | 136.7      | +5%         |

Differential Revision: D94096006
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 28, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 28, 2026

@sammy-SC has exported this pull request. If you are a Meta employee, you can view the originating Diff in D94096006.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants