Skip to content
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.

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

./build.sh libs.native fails #92194

Closed
vargaz opened this issue Sep 17, 2023 · 10 comments · Fixed by #92289
Closed

./build.sh libs.native fails #92194

vargaz opened this issue Sep 17, 2023 · 10 comments · Fixed by #92289
Assignees
Labels
area-Infrastructure-libraries needs-author-action An issue or pull request that requires more info or actions from the author.
Milestone

Comments

@vargaz
Copy link
Contributor

vargaz commented Sep 17, 2023

Description

Running
./build.sh libs.native
fails.

This is with dotnet/runtime master.

Reproduction Steps

Run ./build.sh libs.native

Expected behavior

Succeeds.

Actual behavior

Fails with:

/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error : Object reference not set to an instance of an object. [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at NuGet.Build.Tasks.Console.MSBuildStaticGraphRestore.GetPackageSpec(IMSBuildProject project, IReadOnlyDictionary`2 allInnerNodes) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at NuGet.Build.Tasks.Console.MSBuildStaticGraphRestore.<>c__DisplayClass35_0.<GetDependencyGraphSpec>b__0(ProjectWithInnerNodes project) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error : --- End of stack trace from previous location --- [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.TaskReplicator.Replica.Execute() [/Users/vargaz/git/runtime/Build.proj]

.

Regression?

Yes, this used to work.

Known Workarounds

No response

Configuration

dotnet/runtime d054157 on an apple m1.

Other information

No response

@ghost ghost added the untriaged New issue has not been triaged by the area owner label Sep 17, 2023
@ghost
Copy link

ghost commented Sep 17, 2023

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Description

Running
./build.sh libs.native fails
fails with:

/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error : Object reference not set to an instance of an object. [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at NuGet.Build.Tasks.Console.MSBuildStaticGraphRestore.GetPackageSpec(IMSBuildProject project, IReadOnlyDictionary`2 allInnerNodes) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at NuGet.Build.Tasks.Console.MSBuildStaticGraphRestore.<>c__DisplayClass35_0.<GetDependencyGraphSpec>b__0(ProjectWithInnerNodes project) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error : --- End of stack trace from previous location --- [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.TaskReplicator.Replica.Execute() [/Users/vargaz/git/runtime/Build.proj]

This is with dotnet/runtime master.

Reproduction Steps

.

Expected behavior

.

Actual behavior

.

Regression?

Yes.

Known Workarounds

No response

Configuration

No response

Other information

No response

Author: vargaz
Assignees: -
Labels:

area-Infrastructure-libraries

Milestone: -

@vargaz vargaz removed area-Infrastructure-libraries untriaged New issue has not been triaged by the area owner labels Sep 17, 2023
@vargaz vargaz modified the milestone: 9.0.0 Sep 17, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Sep 17, 2023
@ghost
Copy link

ghost commented Sep 18, 2023

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Description

Running
./build.sh libs.native
fails with:

/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error : Object reference not set to an instance of an object. [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at NuGet.Build.Tasks.Console.MSBuildStaticGraphRestore.GetPackageSpec(IMSBuildProject project, IReadOnlyDictionary`2 allInnerNodes) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at NuGet.Build.Tasks.Console.MSBuildStaticGraphRestore.<>c__DisplayClass35_0.<GetDependencyGraphSpec>b__0(ProjectWithInnerNodes project) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error : --- End of stack trace from previous location --- [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2.<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion) [/Users/vargaz/git/runtime/Build.proj]
/Users/vargaz/git/runtime/.dotnet/sdk/8.0.100-rc.1.23415.11/NuGet.RestoreEx.targets(19,5): error :    at System.Threading.Tasks.TaskReplicator.Replica.Execute() [/Users/vargaz/git/runtime/Build.proj]

This is with dotnet/runtime master.

Reproduction Steps

.

Expected behavior

.

Actual behavior

.

Regression?

Yes.

Known Workarounds

No response

Configuration

No response

Other information

No response

Author: vargaz
Assignees: -
Labels:

area-Infrastructure-libraries, untriaged

Milestone: -

@ViktorHofer
Copy link
Member

Without providing more details, unfortunately not actionable on our side. Could be a NuGet bug.

cc @jeffkl

@ViktorHofer ViktorHofer added needs-author-action An issue or pull request that requires more info or actions from the author. and removed untriaged New issue has not been triaged by the area owner labels Sep 18, 2023
@ViktorHofer ViktorHofer added this to the Future milestone Sep 18, 2023
@ghost
Copy link

ghost commented Sep 18, 2023

This issue has been marked needs-author-action and may be missing some important information.

@jeffkl
Copy link

jeffkl commented Sep 18, 2023

This broke static graph-based restore: faf883d#diff-361b24a1b39d592c236441b48457113d003aeb3bc03df5bbc0318fd0b2e05cd2R17-R18

The way it works is MSBuild walks the graph and tells NuGet about all of the possible instances of a project. For a multi-targeting project, it normally loads it the first time with TargetFrameworks set, then for each value in that list, it loads the project N times with each instance having TargetFramework set. By setting the TargetFramework propety in illink.targets, MSBuild's static graph API will now only load the project for each TargetFramework and not the outer instance. NuGet essentially receives an incomplete representation of the graph. There is no mode in MSBuild to tell it to load the full graph at the moment. This is currently tracked at NuGet/Home#11680

I think if you add to the conditions of the SetTargetFramework to not do that if IsGraphBuild=true will fix it.

@ViktorHofer
Copy link
Member

cc @sbomer

@ViktorHofer
Copy link
Member

ViktorHofer commented Sep 19, 2023

Based on the linked issue title, is the assumption true that in our repo some non SDK style project references the linker projects? If yes, then we could also try to find that non SDK style project.

@jeffkl
Copy link

jeffkl commented Sep 19, 2023

No I think that issue is misleading, the usage of SetTargetFramework is what breaks things. Normally MSBuild and NuGet do target framework negotiation for you by first asking the project what target frameworks it supports and then deciding the "nearest". When you manually provide a value via SetTargetFramework, MSBuild skips that and only loads the project with that TFM. This causes the graph to be incomplete in NuGet's eyes since the outer and all other inner project evaluations are missing.

@ViktorHofer
Copy link
Member

ViktorHofer commented Sep 19, 2023

This causes the graph to be incomplete in NuGet's eyes since the outer and all other inner project evaluations are missing.

As NuGet is only able to restore outer+inner but not inner only, right? At least that's what I saw while digging through the restore code.

@sbomer sbomer self-assigned this Sep 19, 2023
@sbomer
Copy link
Member

sbomer commented Sep 19, 2023

It does look like the failure mode has something to do with ILLink being referenced from a particular project. From the binlogs I see it's happening for native-binplace.proj, during the restore of build-native.proj.

It fails for:

 .dotnet/dotnet msbuild ./.packages/microsoft.dotnet.arcade.sdk/8.0.0-beta.23428.2/tools/Build.proj -p:Restore=true -p:Projects=`pwd`/src/native/libs/build-native.proj

but not

 .dotnet/dotnet msbuild ./.packages/microsoft.dotnet.arcade.sdk/8.0.0-beta.23428.2/tools/Build.proj -p:Restore=true -p:Projects=`pwd`/src/coreclr/System.Private.CoreLib/System.Private.CoreLib.csproj

@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Sep 19, 2023
sbomer added a commit that referenced this issue Sep 20, 2023
Fixes #92194. The
reference to illink from `native-binplace.proj`, built as a
reference of `build-native.proj`, was hitting a nuget bug with
static graph restore. The bug seems to be specific to something
about the project file (maybe the language-specific targets,
since `native-binplace.proj` imports the `Microsoft.NET.Sdk`, but
doesn't have a `csproj` extension).

Fixed by explicitly marking this as not a source project, which
will prevent the import of illink.targets.
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Sep 20, 2023
Artromskiy added a commit to Artromskiy/runtime that referenced this issue Oct 4, 2023
commit a5b75b8
Author: Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Date:   Wed Sep 20 22:04:58 2023 +0200

    JIT: Fix invalid containment of vector broadcasts (dotnet#92333)

    The containment checks for vector broadcasts were missing a size check,
    meaning that a uint broadcast could contain a ubyte/ushort indirection.
    That would lead to out-of-bounds reads.

    Fix dotnet#83387

commit 614d864
Author: Stephen Toub <stoub@microsoft.com>
Date:   Wed Sep 20 15:56:37 2023 -0400

    Use Utf8JsonWriterCache in JsonNode.To{Json}String (dotnet#92358)

commit c0b5150
Author: Andy Gocke <angocke@microsoft.com>
Date:   Wed Sep 20 12:46:37 2023 -0700

    Bring back CopyOutputSymbolsToPublishDirectory (dotnet#92315)

    I accidentally removed this property from AOT compilation
    when adding support for Mac dsym bundles. This change re-enables
    support for suppressing debugging symbols in the output.

    Fixes dotnet#92188

commit b4be77b
Author: Kunal Pathak <Kunal.Pathak@microsoft.com>
Date:   Wed Sep 20 10:17:22 2023 -0700

    Update the assert for BlendVariable (dotnet#92183)

    * Update the assert for BlendVariable

    * Add test cases

    * Add Sse41.IsSupported check

commit e235aef
Author: Miha Zupan <mihazupan.zupan1@gmail.com>
Date:   Wed Sep 20 17:45:31 2023 +0200

    Set severity of rule CA1870 to warning (dotnet#92135)

    * Set severity of rule CA1870 to warning

    * Replace one more usage in nativeaot corelib

    * Set severity for tests as well

    * pragma disable the rule in nativeaot's reflection impl

commit 901f780
Author: Ilona Tomkowicz <32700855+ilonatommy@users.noreply.github.com>
Date:   Wed Sep 20 17:45:01 2023 +0200

    [wasm][debugger] Add tests for indexing by object schema (dotnet#92268)

    * Indexing with object: works.

    * Update expected line numbers.

commit d6ff465
Author: Johan Lorensson <lateralusx.github@gmail.com>
Date:   Wed Sep 20 17:24:39 2023 +0200

    Add missing case for constrained gsharedvt call. (dotnet#92338)

    dotnet@1b788f4
    added a new value to our MonoRgctxInfoType enum type, but appears
    that all cases where not full adjusted. Running System.Buffers tests
    in full AOT hits the assert in info_equal about missing case,
    https://github.com/dotnet/runtime/blob/0dc5903679606b072adac70a268cdb77d1147b3e/src/mono/mono/mini/mini-generic-sharing.c#L2908.

    This commit adds the new enum value and align handling similar to other cases added
    by that commit.

commit 36ab905
Author: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Date:   Wed Sep 20 09:51:37 2023 -0500

    Update dependencies from https://github.com/dotnet/installer build 20230919.3 (dotnet#92339)

    Microsoft.Dotnet.Sdk.Internal
     From Version 9.0.100-alpha.1.23464.17 -> To Version 9.0.100-alpha.1.23469.3

    Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

commit 32c3355
Author: Stephen Toub <stoub@microsoft.com>
Date:   Wed Sep 20 09:42:27 2023 -0400

    Fix downlevel build break in TensorPrimitives (dotnet#92269)

    * Fix downlevel build break in TensorPrimitives

    * Make net6.0 Tensors use ns2.0 implementation

    * Add net6.0 and net7.0 to Tensors temporarily since those are shipping in 8.0 branch

    * Only build net6.0 and net7.0 Tensors when not in source-build

    ---------

    Co-authored-by: Eric StJohn <ericstj@microsoft.com>

commit e8c3052
Author: Matt Thalman <mthalman@microsoft.com>
Date:   Wed Sep 20 07:45:55 2023 -0500

    Update Newtonsoft.Json from 13.0.1 to 13.0.3 (dotnet#92298)

commit b4912a7
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed Sep 20 08:12:52 2023 -0400

    [wasi] Fix llvm target triple. (dotnet#92256)

commit 0dc5903
Author: Artur Zgodziński <bivaro@gmail.com>
Date:   Wed Sep 20 11:45:46 2023 +0100

    Fix trimming of DebuggerDisplay with Name (dotnet#92191)

    The `Name` and `Type` property of the `DebuggerDisplay` attribute accepts the
    same format string as its `Value` property, but does not prevent
    trimming members it references. Thanks to this fix, members referenced by
    any of these two properties are not trimmed and can be displayed by a
    debugger.

commit 521e1e6
Author: Marie Píchová <11718369+ManickaP@users.noreply.github.com>
Date:   Wed Sep 20 12:28:18 2023 +0200

    [QUIC] Throw ODE if connection/listener is disposed (dotnet#92215)

    * AcceptConnection/StreamAsync now throw ODE in case the listener/connection was stopped by DisposeAsync.

    * Fix exception type and make behavior stable for disposal

commit d411f50
Author: Stephen Toub <stoub@microsoft.com>
Date:   Wed Sep 20 06:24:58 2023 -0400

    Avoid unnecessary array allocation in JsonHelpers.Utf8GetString on netstandard (dotnet#92304)

commit 5883b72
Author: Tarek Mahmoud Sayed <tarekms@microsoft.com>
Date:   Tue Sep 19 19:52:38 2023 -0700

    Fix options Validation with objects have indexers (dotnet#92309)

commit fcf7b11
Author: Sven Boemer <sbomer@gmail.com>
Date:   Tue Sep 19 17:51:32 2023 -0700

    Prevent restoring illink for native-binplace.proj (dotnet#92289)

    Fixes dotnet#92194. The
    reference to illink from `native-binplace.proj`, built as a
    reference of `build-native.proj`, was hitting a nuget bug with
    static graph restore. The bug seems to be specific to something
    about the project file (maybe the language-specific targets,
    since `native-binplace.proj` imports the `Microsoft.NET.Sdk`, but
    doesn't have a `csproj` extension).

    Fixed by explicitly marking this as not a source project, which
    will prevent the import of illink.targets.

commit b049f42
Author: Egor Bogatov <egorbo@gmail.com>
Date:   Wed Sep 20 01:39:30 2023 +0200

    Fix optSwitchConvert (dotnet#92249)

    Co-authored-by: Egor <egorbo@Egors-MacBook-Pro.local>

commit 41a8e39
Author: Tanner Gooding <tagoo@outlook.com>
Date:   Tue Sep 19 15:09:19 2023 -0700

    Ensure VN handles both forms of the xarch shift instructions for SIMD (dotnet#91601)

commit 3b9b4fd
Author: Viktor Hofer <viktor.hofer@microsoft.com>
Date:   Tue Sep 19 23:29:29 2023 +0200

    Move portable RID graph into runtime and clean-up (dotnet#92211)

    * Move portable RID graph into runtime and clean-up

    1. Move portable RID graph into runtime
    2. Allow updates to both the non-portable and portable RID graphs under
       source build.
    3. Clean-up project and remove hacks

    * Update README and delete test

    * Fix RID graph update when the key already exists

    * Update src/libraries/Microsoft.NETCore.Platforms/readme.md

    Co-authored-by: Jan Kotas <jkotas@microsoft.com>

    * Update src/libraries/Microsoft.NETCore.Platforms/readme.md

    Co-authored-by: Andy Gocke <angocke@microsoft.com>

    ---------

    Co-authored-by: Jan Kotas <jkotas@microsoft.com>
    Co-authored-by: Andy Gocke <angocke@microsoft.com>

commit 1185d19
Author: Tanner Gooding <tagoo@outlook.com>
Date:   Tue Sep 19 13:41:15 2023 -0700

    Don't generate AddMask as it requires more explicit consideration of semantics (dotnet#92282)

commit a7cafec
Author: Carlos Sánchez López <1175054+carlossanlop@users.noreply.github.com>
Date:   Tue Sep 19 12:14:24 2023 -0700

    [main] Bump Microsoft.Private.IntelliSense package version (dotnet#92255)

commit 094801e
Author: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Date:   Tue Sep 19 12:13:49 2023 -0700

    [main] Update dependencies from dotnet/runtime dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/sdk dotnet/source-build-reference-packages (dotnet#92175)

    * Update dependencies from https://github.com/dotnet/emsdk build 20230915.3

    Microsoft.NET.Workload.Emscripten.Current.Manifest-9.0.100.Transport
     From Version 9.0.0-alpha.1.23457.3 -> To Version 9.0.0-alpha.1.23465.3

    * Update dependencies from https://github.com/dotnet/sdk build 20230915.37

    Microsoft.DotNet.ApiCompat.Task
     From Version 9.0.100-alpha.1.23465.4 -> To Version 9.0.100-alpha.1.23465.37

    * Update dependencies from https://github.com/dotnet/emsdk build 20230915.3

    Microsoft.NET.Workload.Emscripten.Current.Manifest-9.0.100.Transport
     From Version 9.0.0-alpha.1.23457.3 -> To Version 9.0.0-alpha.1.23465.3

    * Update dependencies from https://github.com/dotnet/sdk build 20230916.1

    Microsoft.DotNet.ApiCompat.Task
     From Version 9.0.100-alpha.1.23465.4 -> To Version 9.0.100-alpha.1.23466.1

    * Update dependencies from https://github.com/dotnet/emsdk build 20230915.3

    Microsoft.NET.Workload.Emscripten.Current.Manifest-9.0.100.Transport
     From Version 9.0.0-alpha.1.23457.3 -> To Version 9.0.0-alpha.1.23465.3

    * Update dependencies from https://github.com/dotnet/sdk build 20230918.4

    Microsoft.DotNet.ApiCompat.Task
     From Version 9.0.100-alpha.1.23465.4 -> To Version 9.0.100-alpha.1.23468.4

    * Update dependencies from https://github.com/dotnet/runtime build 20230916.6

    Microsoft.DotNet.ILCompiler , Microsoft.NET.ILLink.Tasks , Microsoft.NET.Sdk.IL , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.ILAsm , runtime.native.System.IO.Ports , System.Text.Json
     From Version 9.0.0-alpha.1.23460.2 -> To Version 9.0.0-alpha.1.23466.6

    * Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230915.1

    Microsoft.SourceBuild.Intermediate.source-build-reference-packages
     From Version 8.0.0-alpha.1.23457.1 -> To Version 9.0.0-alpha.1.23465.1

    * Update dependencies from https://github.com/dotnet/emsdk build 20230915.3

    Microsoft.NET.Workload.Emscripten.Current.Manifest-9.0.100.Transport
     From Version 9.0.0-alpha.1.23457.3 -> To Version 9.0.0-alpha.1.23465.3

    * Update dependencies from https://github.com/dotnet/hotreload-utils build 20230918.2

    Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
     From Version 8.0.0-alpha.0.23461.1 -> To Version 8.0.0-alpha.0.23468.2

    * Update dependencies from https://github.com/dotnet/cecil build 20230918.2

    Microsoft.DotNet.Cecil
     From Version 0.11.4-alpha.23461.1 -> To Version 0.11.4-alpha.23468.2

    * Update dependencies from https://github.com/dotnet/sdk build 20230918.31

    Microsoft.DotNet.ApiCompat.Task
     From Version 9.0.100-alpha.1.23465.4 -> To Version 9.0.100-alpha.1.23468.31

    * Update dependencies from https://github.com/dotnet/source-build-reference-packages build 20230918.3

    Microsoft.SourceBuild.Intermediate.source-build-reference-packages
     From Version 8.0.0-alpha.1.23457.1 -> To Version 9.0.0-alpha.1.23468.3

    ---------

    Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

commit 9bd0e0d
Author: Jeremy Koritzinsky <jekoritz@microsoft.com>
Date:   Tue Sep 19 12:11:38 2023 -0700

    Remove "Is supported on this TFM" logic from marshalling generators and instead handle it during factory construction (dotnet#91768)

    Co-authored-by: Jackson Schuster <36744439+jtschuster@users.noreply.github.com>

commit 17eff3b
Author: Andy Ayers <andya@microsoft.com>
Date:   Tue Sep 19 11:37:48 2023 -0700

    JIT: generalize assert to handle SIMD64 (dotnet#92235)

    Fixes dotnet#91799.

commit 67dbbeb
Author: Andy Ayers <andya@microsoft.com>
Date:   Tue Sep 19 11:30:38 2023 -0700

    JIT: add missing xarch RMW case (dotnet#92252)

    Handle the case where we're indirectly updating a local with a value
    that is not a constant.

    Fixes dotnet#92218.
@ghost ghost locked as resolved and limited conversation to collaborators Oct 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure-libraries needs-author-action An issue or pull request that requires more info or actions from the author.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants