Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

[master] Update dependencies from dotnet/coreclr#36922

Merged
tannergooding merged 6 commits intomasterfrom
darc-master-497eebb2-f99e-499f-9700-3a5b5302ab5d
Apr 17, 2019
Merged

[master] Update dependencies from dotnet/coreclr#36922
tannergooding merged 6 commits intomasterfrom
darc-master-497eebb2-f99e-499f-9700-3a5b5302ab5d

Conversation

@dotnet-maestro
Copy link
Copy Markdown
Contributor

@dotnet-maestro dotnet-maestro Bot commented Apr 16, 2019

This pull request updates the following dependencies

From https://github.com/dotnet/coreclr

  • Build: 20190415.72
  • Date Produced: 4/16/2019 12:43 AM
  • Commit: 47c6539
  • Branch: refs/heads/master
  • Updates:
    • Microsoft.NET.Sdk.IL -> 3.0.0-preview5-27615-72
    • Microsoft.NETCore.ILAsm -> 3.0.0-preview5-27615-72
    • Microsoft.NETCore.Runtime.CoreCLR -> 3.0.0-preview5-27615-72

From https://github.com/dotnet/coreclr

  • Build: 20190416.72
  • Date Produced: 4/17/2019 12:45 AM
  • Commit: f360bbc
  • Branch: refs/heads/master
  • Updates:
    • Microsoft.NET.Sdk.IL -> 3.0.0-preview5-27616-72
    • Microsoft.NETCore.ILAsm -> 3.0.0-preview5-27616-72
    • Microsoft.NETCore.Runtime.CoreCLR -> 3.0.0-preview5-27616-72

…0415.72

- Microsoft.NET.Sdk.IL - 3.0.0-preview5-27615-72
- Microsoft.NETCore.ILAsm - 3.0.0-preview5-27615-72
- Microsoft.NETCore.Runtime.CoreCLR - 3.0.0-preview5-27615-72
@dotnet-maestro
Copy link
Copy Markdown
Contributor Author

dotnet-maestro Bot commented Apr 16, 2019

Auto-Merge Status

This pull request has not been merged because Maestro++ is waiting on the following merge policies.

  • Waiting on checks: corefx-ci, corefx-ci (MacOS x64_Debug), corefx-ci (Linux musl_x64_Release), corefx-ci (Windows UWP_CoreCLR_x64_Debug), corefx-ci (Linux arm64_Release), corefx-ci (Windows NETFX_x86_Release), corefx-ci (Linux x64_Release), corefx-ci (Windows x86_Release), corefx-ci (Windows x64_Debug), corefx-ci (Packaging All Configurations x64_Debug)

@GrabYourPitchforks
Copy link
Copy Markdown
Member

Failing CI tests are perhaps due to dotnet/coreclr#23833? Though I looked over the PR and everything seems good. Still investigating.

@GrabYourPitchforks GrabYourPitchforks self-assigned this Apr 17, 2019
@GrabYourPitchforks
Copy link
Copy Markdown
Member

  ===========================================================================================================
    Discovering: System.Reflection.Emit.Lightweight.Tests (method display = ClassAndMethod, method display options = None)
    Discovered:  System.Reflection.Emit.Lightweight.Tests (found 1 of 38 test case)
    Starting:    System.Reflection.Emit.Lightweight.Tests (parallel test collections = on, max threads = 8)

  Assert failure(PID 14172 [0x0000375c], Thread: 14524 [0x38bc]): FitsIn<unsigned short>(EHSize)

  CORECLR! LCGMethodResolver::GetCodeInfo + 0x479 (0x00007ffe`381f1499)
  CORECLR! getMethodInfoHelper + 0x195B (0x00007ffe`380066ff)
  CORECLR! UnsafeJitFunction + 0x330 (0x00007ffe`37fe02f8)
  CORECLR! MethodDesc::JitCompileCodeLocked + 0x1B4 (0x00007ffe`38067d70)
  CORECLR! MethodDesc::JitCompileCodeLockedEventWrapper + 0x303 (0x00007ffe`380682ab)
  CORECLR! MethodDesc::JitCompileCode + 0x4CF (0x00007ffe`38067adb)
  CORECLR! MethodDesc::PrepareILBasedCode + 0x18E (0x00007ffe`380699ea)
  CORECLR! MethodDesc::PrepareCode + 0xFF (0x00007ffe`3806961b)
  CORECLR! MethodDesc::PrepareInitialCode + 0xEF (0x00007ffe`38069b3f)
  CORECLR! MethodDesc::DoPrestub + 0x837 (0x00007ffe`38062c8f)
      File: C:\coreclr\src\vm\dynamicmethod.cpp Line: 1118
      Image: C:\corefx\artifacts\bin\testhost\netcoreapp-Windows_NT-Debug-x64\dotnet.exe

@GrabYourPitchforks
Copy link
Copy Markdown
Member

The coreclr fix which unblocks this PR will be in the 3.0.0-preview5-27616-73 train.

@dotnet-maestro
Copy link
Copy Markdown
Contributor Author

dotnet-maestro Bot commented Apr 17, 2019

Auto-Merge Status

This pull request has not been merged because Maestro++ is waiting on the following merge policies.

  • Waiting on checks: corefx-ci, corefx-ci (Linux musl_arm64_Release), corefx-ci (MacOS x64_Debug), corefx-ci (Linux arm_Release), corefx-ci (Windows UWP_CoreCLR_x64_Debug), corefx-ci (Linux musl_x64_Release), corefx-ci (Windows NETFX_x86_Release), corefx-ci (Linux arm64_Release), corefx-ci (Windows x86_Release), corefx-ci (Linux x64_Release), corefx-ci (Windows x64_Debug), corefx-ci (Packaging All Configurations x64_Debug)

…0416.72

- Microsoft.NET.Sdk.IL - 3.0.0-preview5-27616-72
- Microsoft.NETCore.ILAsm - 3.0.0-preview5-27616-72
- Microsoft.NETCore.Runtime.CoreCLR - 3.0.0-preview5-27616-72
@safern
Copy link
Copy Markdown
Member

safern commented Apr 17, 2019

@GrabYourPitchforks let me know if you need help doing a manual update or something to get the fix for this PR.

@GrabYourPitchforks
Copy link
Copy Markdown
Member

Spoke with @tannergooding offline. We're cherry-picking #36935 into this PR to unblock the merge.

@tannergooding tannergooding merged commit 818355a into master Apr 17, 2019
@safern safern deleted the darc-master-497eebb2-f99e-499f-9700-3a5b5302ab5d branch April 18, 2019 01:18
@karelz karelz added this to the 3.0 milestone May 4, 2019
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
* Update dependencies from https://github.com/dotnet/coreclr build 20190415.72

- Microsoft.NET.Sdk.IL - 3.0.0-preview5-27615-72
- Microsoft.NETCore.ILAsm - 3.0.0-preview5-27615-72
- Microsoft.NETCore.Runtime.CoreCLR - 3.0.0-preview5-27615-72

* Update dependencies from https://github.com/dotnet/coreclr build 20190416.72

- Microsoft.NET.Sdk.IL - 3.0.0-preview5-27616-72
- Microsoft.NETCore.ILAsm - 3.0.0-preview5-27616-72
- Microsoft.NETCore.Runtime.CoreCLR - 3.0.0-preview5-27616-72

* Update CoreCLR dependencies to 3.0.0-preview5-27616-73

* Fixing the Min/Max and MinMagnitude/MaxMagnitude for System.Math and System.MathF to expect NaN propagation


Commit migrated from dotnet/corefx@818355a
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants