Commit a47f447
committed
[compiler] Add missing source locations to statements, expressions (#34406)
Adds missing locations to all the statement kinds that we produce in
codegenInstruction(), and adds generic handling of source locations for
the nodes produced by codegenInstructionValue(). There are definitely
some places where we are still missing a location, but this should
address some of the known issues we've seen such as missing location on
`throw`.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/34406).
* #34394
* __->__ #34406
* #34346
DiffTrain build for [f5e96b9](f5e96b9)1 parent 118621b commit a47f447
File tree
26 files changed
+203
-169
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
- tools/eslint-plugin-react-hooks
- cjs
26 files changed
+203
-169
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 | | |
| |||
404 | 404 | | |
405 | 405 | | |
406 | 406 | | |
407 | | - | |
| 407 | + | |
408 | 408 | | |
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 | | |
| |||
29580 | 29580 | | |
29581 | 29581 | | |
29582 | 29582 | | |
29583 | | - | |
| 29583 | + | |
29584 | 29584 | | |
29585 | 29585 | | |
29586 | 29586 | | |
29587 | | - | |
| 29587 | + | |
29588 | 29588 | | |
29589 | 29589 | | |
29590 | 29590 | | |
| |||
29621 | 29621 | | |
29622 | 29622 | | |
29623 | 29623 | | |
29624 | | - | |
| 29624 | + | |
29625 | 29625 | | |
29626 | 29626 | | |
29627 | | - | |
| 29627 | + | |
29628 | 29628 | | |
29629 | 29629 | | |
29630 | 29630 | | |
| |||
29773 | 29773 | | |
29774 | 29774 | | |
29775 | 29775 | | |
29776 | | - | |
| 29776 | + | |
29777 | 29777 | | |
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 | | |
| |||
17388 | 17388 | | |
17389 | 17389 | | |
17390 | 17390 | | |
17391 | | - | |
| 17391 | + | |
17392 | 17392 | | |
17393 | 17393 | | |
17394 | 17394 | | |
17395 | 17395 | | |
17396 | 17396 | | |
17397 | 17397 | | |
17398 | | - | |
| 17398 | + | |
17399 | 17399 | | |
17400 | 17400 | | |
17401 | 17401 | | |
| |||
17417 | 17417 | | |
17418 | 17418 | | |
17419 | 17419 | | |
17420 | | - | |
| 17420 | + | |
17421 | 17421 | | |
17422 | 17422 | | |
17423 | | - | |
| 17423 | + | |
17424 | 17424 | | |
17425 | 17425 | | |
17426 | 17426 | | |
| |||
17527 | 17527 | | |
17528 | 17528 | | |
17529 | 17529 | | |
17530 | | - | |
| 17530 | + | |
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 | | |
| |||
19398 | 19398 | | |
19399 | 19399 | | |
19400 | 19400 | | |
19401 | | - | |
| 19401 | + | |
19402 | 19402 | | |
19403 | 19403 | | |
19404 | 19404 | | |
19405 | 19405 | | |
19406 | 19406 | | |
19407 | 19407 | | |
19408 | | - | |
| 19408 | + | |
19409 | 19409 | | |
19410 | 19410 | | |
19411 | 19411 | | |
| |||
19427 | 19427 | | |
19428 | 19428 | | |
19429 | 19429 | | |
19430 | | - | |
| 19430 | + | |
19431 | 19431 | | |
19432 | 19432 | | |
19433 | | - | |
| 19433 | + | |
19434 | 19434 | | |
19435 | 19435 | | |
19436 | 19436 | | |
| |||
19553 | 19553 | | |
19554 | 19554 | | |
19555 | 19555 | | |
19556 | | - | |
| 19556 | + | |
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 | | |
| |||
29636 | 29636 | | |
29637 | 29637 | | |
29638 | 29638 | | |
29639 | | - | |
| 29639 | + | |
29640 | 29640 | | |
29641 | 29641 | | |
29642 | 29642 | | |
29643 | | - | |
| 29643 | + | |
29644 | 29644 | | |
29645 | 29645 | | |
29646 | 29646 | | |
| |||
29677 | 29677 | | |
29678 | 29678 | | |
29679 | 29679 | | |
29680 | | - | |
| 29680 | + | |
29681 | 29681 | | |
29682 | 29682 | | |
29683 | | - | |
| 29683 | + | |
29684 | 29684 | | |
29685 | 29685 | | |
29686 | 29686 | | |
| |||
30145 | 30145 | | |
30146 | 30146 | | |
30147 | 30147 | | |
30148 | | - | |
| 30148 | + | |
30149 | 30149 | | |
30150 | 30150 | | |
30151 | 30151 | | |
| |||
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 | | |
| |||
17399 | 17399 | | |
17400 | 17400 | | |
17401 | 17401 | | |
17402 | | - | |
| 17402 | + | |
17403 | 17403 | | |
17404 | 17404 | | |
17405 | 17405 | | |
17406 | 17406 | | |
17407 | 17407 | | |
17408 | 17408 | | |
17409 | | - | |
| 17409 | + | |
17410 | 17410 | | |
17411 | 17411 | | |
17412 | 17412 | | |
| |||
17428 | 17428 | | |
17429 | 17429 | | |
17430 | 17430 | | |
17431 | | - | |
| 17431 | + | |
17432 | 17432 | | |
17433 | 17433 | | |
17434 | | - | |
| 17434 | + | |
17435 | 17435 | | |
17436 | 17436 | | |
17437 | 17437 | | |
| |||
17691 | 17691 | | |
17692 | 17692 | | |
17693 | 17693 | | |
17694 | | - | |
| 17694 | + | |
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 | | |
| |||
19413 | 19413 | | |
19414 | 19414 | | |
19415 | 19415 | | |
19416 | | - | |
| 19416 | + | |
19417 | 19417 | | |
19418 | 19418 | | |
19419 | 19419 | | |
19420 | 19420 | | |
19421 | 19421 | | |
19422 | 19422 | | |
19423 | | - | |
| 19423 | + | |
19424 | 19424 | | |
19425 | 19425 | | |
19426 | 19426 | | |
| |||
19442 | 19442 | | |
19443 | 19443 | | |
19444 | 19444 | | |
19445 | | - | |
| 19445 | + | |
19446 | 19446 | | |
19447 | 19447 | | |
19448 | | - | |
| 19448 | + | |
19449 | 19449 | | |
19450 | 19450 | | |
19451 | 19451 | | |
| |||
19721 | 19721 | | |
19722 | 19722 | | |
19723 | 19723 | | |
19724 | | - | |
| 19724 | + | |
19725 | 19725 | | |
19726 | 19726 | | |
19727 | 19727 | | |
| |||
0 commit comments