Commit e3cd7a8
committed
Use fastAddProperties in diffing (facebook#32243)
## Summary
`fastAddProperties` has shown some perf benefits when used for creating
props payload for new components. In this PR we'll try to use it for
diffing props for existing components.
It would be good enough if it simply doesn't regress perf. We'll be able
to delete the old `addProperties`, and make `fastAddProperties` the
default behaviour.
## How did you test this change?
```
yarn lint
yarn flow native
yarn test packages/react-native-renderer -r=xplat --variant=false
yarn test packages/react-native-renderer -r=xplat --variant=true
```
DiffTrain build for [bb9a24d](facebook@bb9a24d)1 parent ad3cbd0 commit e3cd7a8
File tree
23 files changed
+106
-104
lines changed- compiled-rn
- facebook-fbsource/xplat/js
- RKJSModules/vendor/react
- react-dom/cjs
- react-test-renderer/cjs
- react/cjs
- react-native-github/Libraries/Renderer
- implementations
23 files changed
+106
-104
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
428 | 428 | | |
429 | 429 | | |
430 | 430 | | |
431 | | - | |
| 431 | + | |
432 | 432 | | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
26013 | 26013 | | |
26014 | 26014 | | |
26015 | 26015 | | |
26016 | | - | |
| 26016 | + | |
26017 | 26017 | | |
26018 | 26018 | | |
26019 | 26019 | | |
26020 | | - | |
| 26020 | + | |
26021 | 26021 | | |
26022 | 26022 | | |
26023 | 26023 | | |
| |||
26054 | 26054 | | |
26055 | 26055 | | |
26056 | 26056 | | |
26057 | | - | |
| 26057 | + | |
26058 | 26058 | | |
26059 | 26059 | | |
26060 | | - | |
| 26060 | + | |
26061 | 26061 | | |
26062 | 26062 | | |
26063 | 26063 | | |
| |||
26203 | 26203 | | |
26204 | 26204 | | |
26205 | 26205 | | |
26206 | | - | |
| 26206 | + | |
26207 | 26207 | | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
16017 | 16017 | | |
16018 | 16018 | | |
16019 | 16019 | | |
16020 | | - | |
| 16020 | + | |
16021 | 16021 | | |
16022 | 16022 | | |
16023 | 16023 | | |
16024 | 16024 | | |
16025 | 16025 | | |
16026 | 16026 | | |
16027 | | - | |
| 16027 | + | |
16028 | 16028 | | |
16029 | 16029 | | |
16030 | 16030 | | |
| |||
16046 | 16046 | | |
16047 | 16047 | | |
16048 | 16048 | | |
16049 | | - | |
| 16049 | + | |
16050 | 16050 | | |
16051 | 16051 | | |
16052 | | - | |
| 16052 | + | |
16053 | 16053 | | |
16054 | 16054 | | |
16055 | 16055 | | |
| |||
16155 | 16155 | | |
16156 | 16156 | | |
16157 | 16157 | | |
16158 | | - | |
| 16158 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
16666 | 16666 | | |
16667 | 16667 | | |
16668 | 16668 | | |
16669 | | - | |
| 16669 | + | |
16670 | 16670 | | |
16671 | 16671 | | |
16672 | 16672 | | |
16673 | 16673 | | |
16674 | 16674 | | |
16675 | 16675 | | |
16676 | | - | |
| 16676 | + | |
16677 | 16677 | | |
16678 | 16678 | | |
16679 | 16679 | | |
| |||
16695 | 16695 | | |
16696 | 16696 | | |
16697 | 16697 | | |
16698 | | - | |
| 16698 | + | |
16699 | 16699 | | |
16700 | 16700 | | |
16701 | | - | |
| 16701 | + | |
16702 | 16702 | | |
16703 | 16703 | | |
16704 | 16704 | | |
| |||
16819 | 16819 | | |
16820 | 16820 | | |
16821 | 16821 | | |
16822 | | - | |
| 16822 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
26074 | 26074 | | |
26075 | 26075 | | |
26076 | 26076 | | |
26077 | | - | |
| 26077 | + | |
26078 | 26078 | | |
26079 | 26079 | | |
26080 | 26080 | | |
26081 | | - | |
| 26081 | + | |
26082 | 26082 | | |
26083 | 26083 | | |
26084 | 26084 | | |
| |||
26115 | 26115 | | |
26116 | 26116 | | |
26117 | 26117 | | |
26118 | | - | |
| 26118 | + | |
26119 | 26119 | | |
26120 | 26120 | | |
26121 | | - | |
| 26121 | + | |
26122 | 26122 | | |
26123 | 26123 | | |
26124 | 26124 | | |
| |||
26580 | 26580 | | |
26581 | 26581 | | |
26582 | 26582 | | |
26583 | | - | |
| 26583 | + | |
26584 | 26584 | | |
26585 | 26585 | | |
26586 | 26586 | | |
| |||
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
16028 | 16028 | | |
16029 | 16029 | | |
16030 | 16030 | | |
16031 | | - | |
| 16031 | + | |
16032 | 16032 | | |
16033 | 16033 | | |
16034 | 16034 | | |
16035 | 16035 | | |
16036 | 16036 | | |
16037 | 16037 | | |
16038 | | - | |
| 16038 | + | |
16039 | 16039 | | |
16040 | 16040 | | |
16041 | 16041 | | |
| |||
16057 | 16057 | | |
16058 | 16058 | | |
16059 | 16059 | | |
16060 | | - | |
| 16060 | + | |
16061 | 16061 | | |
16062 | 16062 | | |
16063 | | - | |
| 16063 | + | |
16064 | 16064 | | |
16065 | 16065 | | |
16066 | 16066 | | |
| |||
16320 | 16320 | | |
16321 | 16321 | | |
16322 | 16322 | | |
16323 | | - | |
| 16323 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
16681 | 16681 | | |
16682 | 16682 | | |
16683 | 16683 | | |
16684 | | - | |
| 16684 | + | |
16685 | 16685 | | |
16686 | 16686 | | |
16687 | 16687 | | |
16688 | 16688 | | |
16689 | 16689 | | |
16690 | 16690 | | |
16691 | | - | |
| 16691 | + | |
16692 | 16692 | | |
16693 | 16693 | | |
16694 | 16694 | | |
| |||
16710 | 16710 | | |
16711 | 16711 | | |
16712 | 16712 | | |
16713 | | - | |
| 16713 | + | |
16714 | 16714 | | |
16715 | 16715 | | |
16716 | | - | |
| 16716 | + | |
16717 | 16717 | | |
16718 | 16718 | | |
16719 | 16719 | | |
| |||
16988 | 16988 | | |
16989 | 16989 | | |
16990 | 16990 | | |
16991 | | - | |
| 16991 | + | |
16992 | 16992 | | |
16993 | 16993 | | |
16994 | 16994 | | |
| |||
0 commit comments