Skip to content

Conversation

@dotnet-bot
Copy link
Contributor

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

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

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

TIHan and others added 7 commits July 9, 2019 19:47
…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
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approval

@ghost ghost merged commit b6f0be9 into release/fsharp47 Jul 11, 2019
@ghost ghost deleted the merges/release/dev16.3-to-release/fsharp47 branch July 11, 2019 07:01
This pull request was closed.
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.

4 participants