Skip to content

Conversation

@dotnet-bot
Copy link
Contributor

This is an automatically generated pull request from master into release/dev16.3.
(created from a manual run of the PR generation tool)

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

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

dotnet-maestro and others added 3 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
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 7d5f12d into release/dev16.3 Jul 10, 2019
@ghost ghost deleted the merges/master-to-release/dev16.3 branch July 10, 2019 23:31
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