forked fromfsharp/fsharp
- Notifications
You must be signed in to change notification settings - Fork0
Commit32a93aa
authored
Ref scoping (#5240)
* Added Limit* Trying to get basic scopes* Basic scope checks* Some more checks* Handling another ref scope case* Using env instead of cenv* Partially scoped* Almost figuring out scoping* Scopes are working* Check returnable* Current tests passing* Removed compiler generated rhs* Minor cleanup* Added scoping tests* Adding tests again* Scoping now determines what is local* Updated tests/baseline for byrefs. Simplified compiler generated check* Trivial change for scope check* Fixed bug with stack referring span-likes returning by-refs in scopes other than the method/function level* Quick fix* Preventing taking address of expression that isn't a let-bound value* Updated baseline* Updated baseline* Added better way to disable address of* Finishing up. Updating baselines* Updating neg106 baseline* Updated baselines again* Removing compiler generated check by fixing GetLimitVal1 parent76dd383 commit32a93aa
File tree
31 files changed
+521
-135
lines changed- src
- buildfromsource/FSharp.Compiler.Private
- fsharp
- xlf
- tests/fsharp
- core
- byrefs
- span
- typecheck/sigs
31 files changed
+521
-135
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4351 | 4351 | | |
4352 | 4352 | | |
4353 | 4353 | | |
| 4354 | + | |
| 4355 | + | |
| 4356 | + | |
4354 | 4357 | | |
4355 | 4358 | | |
4356 | 4359 | | |
| |||
5766 | 5769 | | |
5767 | 5770 | | |
5768 | 5771 | | |
| 5772 | + | |
5769 | 5773 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4354 | 4354 | | |
4355 | 4355 | | |
4356 | 4356 | | |
| 4357 | + | |
| 4358 | + | |
| 4359 | + | |
4357 | 4360 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1438 | 1438 | | |
1439 | 1439 | | |
1440 | 1440 | | |
1441 | | - | |
| 1441 | + | |
| 1442 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
| 137 | + | |
137 | 138 | | |
138 | 139 | | |
139 | 140 | | |
140 | 141 | | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
141 | 145 | | |
142 | 146 | | |
143 | 147 | | |
| |||
0 commit comments
Comments
(0)