Skip to content

Span for .. in .. do optimization #6213

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
KevinRansom merged 27 commits into from
Apr 16, 2019
Merged

Span for .. in .. do optimization #6213

Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
7b7dc8f
Trying to optimize span in for loop
TIHan Feb 5, 2019
d064f9b
Added Span_GetItem call
TIHan Feb 7, 2019
84bc179
Almost success with optimization
TIHan Feb 7, 2019
e088062
Fixed code gen
TIHan Feb 8, 2019
343d29d
Added Span optimization tests
TIHan Feb 11, 2019
4d5f2b9
Added ReadOnlySpan opt
TIHan Feb 12, 2019
2b06a30
Cleaning up tests
TIHan Feb 14, 2019
123dcd1
Moving tests around
TIHan Feb 15, 2019
a483901
Trying to figure out span tests
TIHan Feb 15, 2019
2931b10
Trying to fix some tests
TIHan Feb 19, 2019
4a09465
Trying to get some more tests passing
TIHan Mar 11, 2019
b506e37
Fixed range
TIHan Mar 11, 2019
185e86f
Trying to get tests to pass again
TIHan Mar 12, 2019
b331303
Fixing tests for netcore
TIHan Mar 12, 2019
3a8829a
Fix build
TIHan Mar 12, 2019
318f875
When a solution becomes unloaded, we should clear F#'s cache (#6420)
TIHan Apr 3, 2019
46f1b6d
Changing if directives
TIHan Apr 5, 2019
d7352d3
Simplifying
TIHan Apr 5, 2019
35c755a
Using a type shape for span optimization
TIHan Apr 10, 2019
ae91449
Fixing one test
TIHan Apr 10, 2019
d233338
Drastically simplified looking at the type shape for Span
TIHan Apr 10, 2019
962c634
Simplified a bit more
TIHan Apr 10, 2019
f2e950c
RunScript has expected error messages
TIHan Apr 10, 2019
4b5c002
Add back net472
TIHan Apr 10, 2019
76e30d5
Feedback
TIHan Apr 11, 2019
3764d52
Update SpanOptimizationTests.fs
TIHan Apr 11, 2019
b7a886a
Update SpanOptimizationTests.fs
TIHan Apr 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update SpanOptimizationTests.fs
  • Loading branch information
@TIHan
TIHan authored Apr 11, 2019
commit b7a886aa5d798d3cae8edf3ea0e4a573ca479de4
6 changes: 3 additions & 3 deletions tests/fsharp/Compiler/Language/SpanOptimizationTests.fs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ let test () =
"""

ILChecker.checkWithDlls
(ILChecker.getPackageDlls "System.Memory" "4.5.0" "netstandard2.0" [ "System.Memory.dll" ])
(ILChecker.getPackageDlls "System.Memory" "4.5.2" "netstandard2.0" [ "System.Memory.dll" ])
source
[
""".method public static void test() cil managed
Expand Down Expand Up @@ -83,7 +83,7 @@ let test () =
"""

ILChecker.checkWithDlls
(ILChecker.getPackageDlls "System.Memory" "4.5.0" "netstandard2.0" [ "System.Memory.dll" ])
(ILChecker.getPackageDlls "System.Memory" "4.5.2" "netstandard2.0" [ "System.Memory.dll" ])
source
[
""".method public static void test() cil managed
Expand Down Expand Up @@ -171,7 +171,7 @@ module Test =

// The current behavior doesn't optimize, but it could in the future. Making a test to catch if it ever does.
ILChecker.checkWithDlls
(ILChecker.getPackageDlls "System.Memory" "4.5.0" "netstandard2.0" [ "System.Memory.dll" ])
(ILChecker.getPackageDlls "System.Memory" "4.5.2" "netstandard2.0" [ "System.Memory.dll" ])
source
[
""".method public static void test() cil managed
Expand Down