Skip to content

Conversation

@dotnet-bot
Copy link
Contributor

This is an automatically generated pull request from release/dev16.3 into release/fsharp5.

git fetch --all
git checkout merges/release/dev16.3-to-release/fsharp5
git reset --hard upstream/release/fsharp5
git merge upstream/release/dev16.3
# Fix merge conflicts
git commit
git push upstream merges/release/dev16.3-to-release/fsharp5 --force

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request.

dotnet-maestro and others added 5 commits July 10, 2019 12:57
…709.6

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19359.6
* Trying to fix the stack overflow case in ilxgen

* Added stack guard to GenExpr

* Still trying to resolve this

* This resolves the stack overflow, but need to review

* Adding better tests

* Fixing compile and run successfully assert

* One test passes

* Fixing a few things

* Delete fieldNames.txt

* Update IlxGen.fs

* Update IlxGen.fs

* Update IlxGen.fs

* More work on CompilerAssert

* Got verifying IL working on method

* Always delay gen methods when already gen'ing in one. Some small refactoring, created IlxMethodInfo type instead of using large tuples.

* Better error message for stack overflow

* small msg change

* Simplifying some tests

* Removing attempt at bringing a FSharpQA test to nunit

* Fixing up seq points

* More seq point tweaks

* Trying to fix ilxgen

* Fixing ilxgen

* Using RunScript

* Fix test

* removed IlxMethodInfo

* trivial refactor

* another quick refactor
…a13-425ed155cfac

[master] Update dependencies from dotnet/arcade
@dotnet-bot dotnet-bot force-pushed the merges/release/dev16.3-to-release/fsharp5 branch from 1023d1d to 7d5f12d Compare July 11, 2019 12:06
@v-asdhin
Copy link

@brettfo There is a file conflict. Please have a look.

@brettfo
Copy link
Member

brettfo commented Jul 11, 2019

@TIHan can you resolve this? It looks related to #7151.

@KevinRansom KevinRansom merged commit 7d5f12d into release/fsharp5 Jul 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants