JIT: build pred lists before patchpoint expansion (#81196)

Move pred list building 3 phases earlier. It now happens just after
instrumentation (or importation if we're not instrumenting) and just before
expanding patchpoints.

Revise the patchpoint, indirect call, and post importer cleanup phases
to do proper pred list maintenance.

Update the flow checker to handle cases we see when we haven't yet run
the post importer cleanup.

Contributes to #80193.

Use RunContinuationsAsynchronously for block completion tasks (#81013)

* Use RunContinuationsAsynchronously for block completion tasks

* Tweak test

Co-authored-by: Petr Onderka <gsvick@gmail.com>

Fix Configuration Binding with Instantiated Objects (#81050)

[mono][wasm] Avoid saving the result of OP_AOTCONST into the gc pin area, its not needed. (#81179)

Disable the /JIT/opt/ObjectStackAllocation/ObjectStackAllocationTests (#81195)

* Disable the /JIT/opt/ObjectStackAllocation/ObjectStackAllocationTests

Disable the test until a dotnet with a fix of the issue #81103 is used
in the runtime repo to build / run tests.

* Moving the disabling to crossgen2 specific section

Fix a variety of issues in FrozenCollections (#81194)

- Add the new ToFrozenDictionary/Set methods that take optimizedForReading and make the existing overloads cheaper for construction.  This makes construction time and allocation by default much closer to that of a normal dictionary or set while also enabling more of the implementation to be trimmed away by default.
- Fixed the use of FrozenSet/Dictionary.Empty such that we only use the singleton when its comparer matches; otherwise, we could end up handing back an instance with the wrong comparer.
- Avoided passing both a dictionary/set and a comparer down through constructors, as the dictionary/set contains the comparer and we don't want to accidentally end up in a situation where we pass through the wrong one.
- Condensed some of the construction code to be a bit less verbose
- Removed unused locals from ToFrozenDictionary (the min/max/range were no longer being used)
- Remove unnecessary array allocations from PickIntegerDictionary
- Delete the sparse range integer sets, which are rarely needed and are broken for TryGetValue
- Fix CheckUniqueAndUnfoundElements to slice down to only the relevant size prior to clearing

Update dependencies from https://github.com/dotnet/arcade build 20230125.2 (#81213)

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23074.2 -> To Version 8.0.0-beta.23075.2

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

[main] Update dependencies from 9 repositories (#80916)

* Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20230120.1

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.3.5-beta1.23068.2 -> To Version 3.3.5-beta1.23070.1

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

Microsoft.NET.Workload.Emscripten.net7.Manifest-8.0.100
 From Version 8.0.0-alpha.1.23066.1 -> To Version 8.0.0-alpha.1.23070.1

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

Microsoft.NET.Sdk.IL , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.ILAsm , runtime.native.System.IO.Ports , System.Text.Json
 From Version 8.0.0-alpha.1.23065.5 -> To Version 8.0.0-alpha.1.23072.5

* Update dependencies from https://github.com/dotnet/llvm-project build 20230123.1

runtime.linux-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.osx.11.0-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.osx.11.0-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.win-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.win-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.win-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.ObjWriter
 From Version 1.0.0-alpha.1.23066.1 -> To Version 1.0.0-alpha.1.23073.1

* Update Versions.props

fix previos merge

* Update dependencies from https://github.com/dotnet/icu build 20230123.1

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 8.0.0-alpha.1.23066.1 -> To Version 8.0.0-alpha.1.23073.1

* Update dependencies from https://github.com/dotnet/xharness build 20230123.2

Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit
 From Version 1.0.0-prerelease.23068.2 -> To Version 1.0.0-prerelease.23073.2

* Update dependencies from https://github.com/dotnet/runtime-assets build 20230123.1

Microsoft.DotNet.CilStrip.Sources , System.ComponentModel.TypeConverter.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 8.0.0-beta.23066.2 -> To Version 8.0.0-beta.23073.1

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

Microsoft.NET.Workload.Emscripten.net7.Manifest-8.0.100
 From Version 8.0.0-alpha.1.23066.1 -> To Version 8.0.0-alpha.1.23073.2

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

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 1.1.0-alpha.0.23066.1 -> To Version 1.1.0-alpha.0.23073.1

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

Microsoft.DotNet.Cecil , Microsoft.DotNet.Cecil.Pdb
 From Version 0.11.4-alpha.23067.1 -> To Version 0.11.4-alpha.23073.1

* Update dependencies from https://github.com/dotnet/llvm-project build 20230124.1

runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools
 From Version 14.0.0-alpha.1.23066.2 -> To Version 14.0.0-alpha.1.23074.1

* Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20230124.3

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.3.5-beta1.23068.2 -> To Version 3.3.5-beta1.23074.3

* Update dependencies from https://github.com/dotnet/icu build 20230126.1

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 8.0.0-alpha.1.23066.1 -> To Version 8.0.0-alpha.1.23076.1

* Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20230126.1

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.3.5-beta1.23068.2 -> To Version 3.3.5-beta1.23076.1

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

Fix max chunk size limiting (#81192)

When the new stub heaps were implemented, the chunk size needed to be
limited so that all precodes for a chunk fit into a single memory page.
That was done in `MethodDescChunk::CreateChunk`. But it was discovered
now that there is another place where it needs to be limited, the
`MethodTableBuilder::AllocAndInitMethodDescChunk`. The
JIT\opt\ObjectStackAllocation\ObjectStackAllocationTests started to fail
in the outerloop due to too long chunk. The failure happens in crossgen2
as it is using a separate build of runtime in release and only that
uncovers the problem. And only when DOTNET_TieredCompilation=0 and
DOTNET_ProfApi_RejitOnAttach=0.

This change fixes the problem. With this change applied to the dotnet
runtime version used by the crossgen2 and patching it in place in the
.dotnet/shared/.... directory, the issue doesn't occur.

Close #81103

[mono] Fix missing simd mul/div case (#81201)

* Test missing case

* Update src/mono/mono/mini/simd-intrinsics.c

* Update src/mono/mono/mini/simd-intrinsics.c

[wasm] Always allocate when computing temp file name (#81085)

The wasm implementation of __randname uses pointer address as another entry into the randomness.

[mono][interp] Extract code outside of interp main loop (#81208)

Add SYSLIB* analyzers to global configs (#80930)

* Add SYSLIB* analyzers to global configs

* Address PR feedback

Add Lerp to IFloatingPointIeee754 (#81186)

* Add Lerp to IFloatingPointIeee754

* Add tests covering Lerp

* Fix build error

* Handle negative zero

Fix GC interfaces versioning (#81188)

* Fix GC interfaces versioning

The change that introduced GC interfaces versioning had a bug preventing
it from using .NET 8 GC being used with .NET 7 runtime. The
GC_VersionInfo return value should have been the minimum supported
version, not the current one. But more importantly there was also
some confusion on what interface the GC_INTERFACE_MAJOR_VERSION
represents. While the change considered it to be a version of the
IGCToCLR interface, it really means the version of the IGCHeap
interface. This change creates a separate version,
EE_INTERFACE_MAJOR_VERSION for versioning o the IGCToCLR interface to
rectify that.
@Maoni0 also didn't like the way of creating a new version of
IGCToCLR interface for each major version change, so I am changing it to
single IGCToCLR interface.

* Fix version check at the call site of the LogErrorToHost

Fix CA2021 in MethodBodyScanner (#81200)

Updating JitInterface to support Vector512. (#81197)

* Auto-generated changes for adding Vector512 in JitInterface.

* Additional Interface Changes.

* Updating JITEEVersionIdentifier.

BrowserDebugProxy: unify debug metadata reading for PE and Webcil (#81099)

* DebugStore: factor common PE and Webcil reading logic

* Move common logic to a MetadataDebugSummary class

Also switch from cascade of 'if's to a 'switch' when looking at debug entries

* Implement PDB checksum reader for WebcilReader

* Move WebcilReader reflection to a helper; add lazy initialization

Co-authored-by: Ankit Jain <radical@gmail.com>

Update dependencies from https://github.com/dotnet/arcade build 20230124.2 (#81161)

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23073.10 -> To Version 8.0.0-beta.23074.2

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

Update the JIT to track `Span.Length` and `ReadOnlySpan.Length` as "never negative" (#81055)

* Update the JIT to track `Span.Length` and `ReadOnlySpan.Length` as "never negative"

* Updating the "is never negative" info to be tracked in LclVarDsc

* Apply formatting patch

* Ensure lvIsNeverNegative is propagated to shadows

* Update src/coreclr/jit/gentree.h

Co-authored-by: SingleAccretion <62474226+SingleAccretion@users.noreply.github.com>
* Responding to PR feedback

* Adding two asserts

* Rename GenTreeField::IsNeverNegative to IsSpanLength and add a comment

* Fix an assert

Co-authored-by: SingleAccretion <62474226+SingleAccretion@users.noreply.github.com>

avoid allocation of SelectClientCertificate delegate (#81096)

* avoid allocation of SelectClientCertificate delegate

* cleanup

[mono] Build mono logging profiler on s390x (#72049)

At the moment we don't have too many profiling options on s390x, and
the logging profiler comes in handy. Enable building it on s390x and
define MONO_DLL_EXPORT in order to unhide mono_profiler_init_log().

Change banana build to Release (#81127)

Works around https://github.com/dotnet/runtime/issues/79561

Beef up `WithoutFunctionPointerType` (#81147)

We don't have a `MethodTable` shape for function pointer types because how they should look like hasn't been defined until .NET 8.

Beef up the CoreCLR compat shim. We'll not be 100% compatible with this because `typeof(Tuple<delegate*<void>[]>)` doesn't actually load as `typeof(Tuple<IntPtr[]>)`, but this'll do until #71883.

Fix PAL exception SXS test (#81169)

The test was failing in the outerloop CI testing because of a bug in the
test. One global variable that was modified by the main and checked by
the SIGSEGV handler was not marked as volatile.

Close #81113

Add missing case for upper save/restore (#81091)

* Add missing case for upper save/restore

* PR review feedback:

[wasm] Disable ExceptionTests due to issue #81170, to get CI green (#81171)

Issue #81170

Allow forced building of mobile manifest pkgs, outside mobile builds (#80920)

Fix generic parameter data flow validation in NativeAOT (#80956)

Previously generic data flow was done from generic dictionary nodes. Problem with that approach is that there's no origin information at that point. The warnings can't point to the place where the problematic instantiation is in the code - we only know that it exists.
Aside from it being unfriendly for the users, it means any RUC or suppressions don't work on these warnings the same way they do in linker/analyzer.

This change modifies the logic to tag the method as "needs data flow" whenever we spot an instantiation of an annotated generic in it somewhere.
Then the actualy validation/marking is done from data flow using the trim analysis patterns.

The only exception to this is generic data flow for base types and interface implementations, that one is done on the EEType nodes.

Note that AOT implements a much more precise version of the generic data flow validation as compared to linker/analyzer. See the big comment at the bening of GenericParameterWarningLocation.cs for how that works.

Changes the expected warning validation to use tokens to compare message origins (same reason as with Kept validation - consistently converting things to string is hard)

Adds a new dependency node with the generic type definition to the graph and then does analysis on that node.
This is to avoid potential noise warnings which could happen due to multiple instantiations calling the checking code multiple times. With this the check is done only once on the type definition.

Tweaked some tests to try to cover the multiple instances scenario.

Add Utf8JsonWriter.WriteRawValue(System.Buffers.ReadOnlySequence). (#76444)

* Add Utf8JsonWriter.WriteRawValue(System.Buffers.ReadOnlySequence).

* Fix comment: remove TODO; rename GrowIfNeeded().

* Fix comment: consider ReadOnlySequence.Length >= int.Max.

* Make test case more generic for long typed length test data.

* Fix comment: add back TODO issue; checked cast long length to int at begin.

* Fix comment: revert checked cast long length.

Ensure converter factories do not throw on unsupported types. (#81133)

* Ensure converter factories do not throw on unsupported types.

* Update src/libraries/System.Text.Json/tests/System.Text.Json.FSharp.Tests/UnionTests.fs

* Properly implement the Multi-dimensional array converter in tests.

JIT: Fix tailcall-to-loop improper locals zeroing (#81083)

The zeroing that the tailcall-to-loop optimization does was zeroing the
promoted copies for implicit byrefs even when promotion of them was undone.
This was introducing unexpected references to the promoted fields.

Fix #81081

Re-enable ApiCompat ref<->src assembly validation (#81104)

* Re-eanble ApiCompat ref<->src assembly validation

* Fix API compatibility differences and update ApiCompat baseline

* Fix mono build

* Disable ApiCompat for linker

[wasm] Enable Vector4.Dot (#81130)

* [wasm] Enable Vector4.Dot

We have all the pieces already in place

* fix condition

Co-authored-by: Larry Ewing <lewing@microsoft.com>

Simplify getting full names for type loader purposes (#81069)

Avoid constraint validation (#81068)

Avoids bringing ConstructorInfo to simple apps.

[wasm] Improve jiterpreter jit calls (#81071)

* Misc. cleanups and jiterpreter refactorings
* Optimize the ldloca + tnn.load pair in direct jit calls
* Fix argOffsets size calculation
* Turn stats back off

[main] Update dependencies from dotnet/arcade (#81079)

* Update dependencies from https://github.com/dotnet/arcade build 20230123.10

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23067.5 -> To Version 8.0.0-beta.23073.10

* Update tests.proj

In reaction to https://github.com/dotnet/arcade/commit/20e5e15262b7666d9e7b22bdc7ec3fb3d5e67a8f

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

[main] Update dependencies from dotnet/roslyn (#80997)

* Update dependencies from https://github.com/dotnet/roslyn build 20230121.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23070.10 -> To Version 4.6.0-1.23071.1

* Update dependencies from https://github.com/dotnet/roslyn build 20230123.4

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23070.10 -> To Version 4.6.0-1.23073.4

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

Move mono llvm fullaot runtime test leg to rolling build (#81125)

Since it takes quite a while to complete this leg, we should move it off of running every PR.

Addresses #65626

Fix TarWriter.TarEntry exception when adding file whose Unix group owner does not exist in the system (#81070)

* Do not throw if a unix group is non-existent when creating a TarEntry from a file using TarWriter.

* Adjust test that was consuming removed interop method.

* Add unit tests to add file entry whose group owner does not exist.

* Address src feedback.

* Address test feedback

* Delay creation/deletion of group in test

* Test all PosixTarEntry formats

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>

[wasm] Fix S.N.Vector4 related assert (#81126)

Fixes

    error : Precompiling failed for /root/helix/work/workitem/e/wbt/blazorwasm_Release_aot_mv10loow.lgf/obj/Release/net8.0/wasm/for-publish/aot-in/aot-instances.dll with exit code 134. [/root/helix/work/workitem/e/wbt/blazorwasm_Release_aot_mv10loow.lgf/blazorwasm_Release_aot_mv10loow.lgf.csproj]
    error :   %6 = load %System.Numerics.Vector4.15073, %System.Numerics.Vector4.15073* %5, align 1 [/root/helix/work/workitem/e/wbt/blazorwasm_Release_aot_mv10loow.lgf/blazorwasm_Release_aot_mv10loow.lgf.csproj]
    error : <4 x float> [/root/helix/work/workitem/e/wbt/blazorwasm_Release_aot_mv10loow.lgf/blazorwasm_Release_aot_mv10loow.lgf.csproj]
    error : * Assertion: should not be reached at /__w/1/s/src/mono/mono/mini/mini-llvm.c:1455

where we were getting wrong llvm type for Vector4

[wasm] Fix type check for PackedSimd.Splat (#81100)

The Splat's 1st parameter is not Vector128'1, but element type.

Fixes this assertion:

    converting llvm method System.Runtime.Intrinsics.Vector128`1<sbyte> System.Runtime.Intrinsics.Wasm.PackedSimd:Splat (sbyte)
    * Assertion at D:\git\wa-local\src\mono\mono\mini\simd-intrinsics.c:690, condition `vector_type->type == MONO_TYPE_GENERICINST' not met

JIT: build pred lists before inlining (#81000)

Move pred list building up a bit further. Note that this impacts both the root
method and all inlinees, since we run a number of the initial phases on both.

Since inlinees build their basic blocks and flow edges in the root compiler's
memory pool, the only work required to unify the pred lists for a successful
inline is to get things right at the boundaries. And for failed inlines there
is no cross-referencing so we can just let the new pred lists leak away (like
we already do for the inlinee blocks).

Contributes towards #80193.

Disable eventsourceerror test (#81094)

[browser] improve default initial memory size (#80507)

make EmccInitialHeapSize dynamic based on size of the assemblies and AOT DATA segments

Co-authored-by: Ankit Jain <radical@gmail.com>

[wasm] ManagedToNativeGenerator: Skip unmanaged dlls (#81066)

* [wasm] ManagedToNativeGenerator: Skip unmanaged dlls

.. instead crashing with an exception like:

```
src/mono/wasm/build/WasmApp.Native.targets(296,5): error MSB4018: (NETCORE_ENGINEERING_TELEMETRY=Build) The "ManagedToNativeGenerator" task failed unexpectedly.
System.BadImageFormatException: This PE image is not a managed executable.
   at System.Reflection.MetadataLoadContext.LoadFromStreamCore(Stream peStream)
   at System.Reflection.MetadataLoadContext.LoadFromAssemblyPath(String assemblyPath)
   at PInvokeTableGenerator.Generate(String[] pinvokeModules, String[] assemblies, String outputPath) in /_/src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs:line 42
   at ManagedToNativeGenerator.ExecuteInternal() in /_/src/tasks/WasmAppBuilder/ManagedToNativeGenerator.cs:line 68
   at ManagedToNativeGenerator.Execute() in /_/src/tasks/WasmAppBuilder/ManagedToNativeGenerator.cs:line 53
   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
   at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask)
```

The wasm targets currently are not able to differentiate the managed
assemblies from the unmanaged ones. so it needs to be handled here.

* [wasm] WasmAppHost: Add support for host arguments

* Drop reallocation of managed dlls.

Co-authored-by: Marek Fišera <mara@neptuo.com>

Fix a few static linking issues (#81086)

* Fix a few static linking issues

* Dedup paths

* Update src/coreclr/nativeaot/BuildIntegration/Microsoft.NETCore.Native.Unix.targets

Co-authored-by: Adeel Mujahid <3840695+am11@users.noreply.github.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>

[Android] Bump target SDK version in Android manifest (#80923)

* Set target SDK version

* Update Android docs

* Disable tests that do not pass with target API 31

* Check if the installed Android SDK is up-to-date

* Update skip explanation

* Use latest SDK

* Disable failing System.Net.Security test

Minor cleanup in frozen collection code (#81080)

Co-authored-by: Martin Taillefer <mataille@microsoft.com>

Fixed assertion for targetType and op1 type not being the same (#81060)

Fix publishing to Arcade when there are no merged tests in the set (#81054)

Optimize "new DateTime(<const args>)" via improvements in JIT VN (#81005)

Co-authored-by: SingleAccretion <62474226+SingleAccretion@users.noreply.github.com>

[Android] Add option to skip generating apk for a specific project (#80909)

* Add option to skip generating apk for a specific project

* Replace _SkipAndroidGenerateAppBundle with AndroidGenerateAppBundle

[mono][interp] Remove dependence on svar/dvar when computing call offset (#81017)

* [mono][interp] Rename field to better reflect its content

* [mono][interp] Fix stack alignment

* [mono][interp] Move assertion to catch misalignment sooner

* [mono][interp] Remove dependence on svar/dvar when computing call offset

On optimized code, we made sure we always had a svar set, even if the call had no args. We then resolved the offset of the call as the offset of the first arg.
On unoptimized code, we made sure we always had a dvar set, even if the call has void return. We then resolved the offset of the call as the offset of the dvar. As we are introducing alignment guarantees of param area and, in the future, the dvar (only if it is simd type), the offset of the dvar and svar can end up with quite different offsets.

This commit solves these complications by introducing a call_offset field in the call info data, which represents the offset of the call args. This field is set either by the var offset allocator or on the fly, during compilation, in unoptimized case. The dvar offset is allocated independently and when emitting compacted instruction for calls, we do it now in an unified way between optimized and unoptimized compilation.

* Enable assertion temporarily for CI

Speed up frozen string dictionaries and sets by ~25-30% (#81021)

- Replace the specialized comparer types with specialized dictionary
and set types, to avoid virtual dispatch overhead and enable better
inlining.

Co-authored-by: Martin Taillefer <mataille@microsoft.com>

JIT: Add Statement::m_treeListEnd (#81031)

Before this change statements have only one field for the "tree list"
even though bidirectional iteration is supported after nodes have been
linked. This worked fine before the locals tree list existed since the
"root node" was always the last node.

This does not work for the locals tree list since the "root node" is not
part of the list, meaning that we need somewhere else to store the
'last' node. Before this PR the assumption was that the root node was
_never_ part of the locals linked list, so we could use the
gtNext/gtPrev fields of the root node. However, the added test case
shows that in fact it is possible we end up with a top level local.

This PR fixes the problem by adding a Statement::m_treeListEnd field
that can keep the last node of the locals linked list.  While this takes
some memory, it seems like the most maintainable way to resolve the
problem. I experimented with making the linked list circular or by
allocating a new stand-in node when necessary but eventually I ended up
here.

Fix #81018

[iOS][Android] ActiveIssue SpanTests and Matrix4x4Tests that are failing on CI (#80877)

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

[wasm] Add direct jit calls to jiterpreter (#80968)

* Introduce 'direct jit call' mode where jiterpreter compiled trampolines invoke the target directly instead of using gsharedvt wrappers
* Fix jitcall target cache being keyed on the wrapper instead of the actual call target

Uncover IlcGenerateMstatFile (#80998)

[webcil] Minor cleanups (#81009)

* replace magic constants by strlen(LITERAL)

   Let the C compiler do the math

* Use CopyIfDifferent for webcil converter in WasmAppBuilder

* WasmAppHost: serve .webcil as application/octet-stream

* Add mime types for webcil in two more places

* Always add to _fileWrites

   But log whether we actually copied anything or not

Co-authored-by: Ankit Jain <radical@gmail.com>

removed RWLock (#81058)

[browser] benchmark fix aggressive trimming (#81037)

* fix trimming

* revert https://github.com/dotnet/runtime/pull/81027

* Fix build

Co-authored-by: Radek Doulik <radekdoulik@gmail.com>

[wasm][debugger] Implement support to symbolOptions from dap. (#79284)

* Draft to implement support to symbolOptions from dap.

* removing debugger.launch

* Adding tests and fix compilation error

* Adding test case.

* Fix test cases, and implement support to PDBChecksum used on nuget.org to get symbols.

* merge

* Fixing tests.

* Tests are timing out.

* Apply suggestions from code review

Co-authored-by: Larry Ewing <lewing@microsoft.com>
Co-authored-by: Ankit Jain <radical@gmail.com>
* adressing @radical comments.

* Addressing @radical comment.

* Addressing @radical comments.

* Apply suggestions from code review

Co-authored-by: Ankit Jain <radical@gmail.com>
* Addressing @radical comments.

* Addressing @radical comments
Changing when the symbols from symbol server is loaded because it takes a long time to load, as there are a lot of assemblies loaded not found on symbol servers.

* use MicrosoftCodeAnalysisCSharpVersion for scripting package.

* Adding more tests as asked by @radical
Removing timeout change as @radical has split it into 2 files
Fixing test behavior, when justMyCode is disabled but the symbols are not loaded from symbol server.

* [wasm] some cleanup

- Don't call `UpdateSymbolStore` from `DebugStore..ctor` because that
gets called multiple times in `LoadStore`, but only once instance gets
used.
- Use an isolated symbol cache path per test

* remove debug spew

* Addressing radical comment.

Co-authored-by: Larry Ewing <lewing@microsoft.com>
Co-authored-by: Ankit Jain <radical@gmail.com>

[Mono] Fix function pointer check (#80855)

* Fix function pointer check

* Make is_monomorphic_array return false when the element is function pointer

* Move test to a better location

Add expected output tests for regex source generator (#80937)

* Add expected output tests for regex

In particular to help avoid significant formatting / readability regressions in the code output by the regex generator.

* Address PR feedback to normalize version

[mono][aot] Free LLVM module after AOT. (#81014)

This is needed to avoid out of memory errors when multiple assemblies
are AOT-ed during dedup.

Update suppressions now that netframework facade has forwards (#79465)

* Update suppressions now that netframework facade has forwards

* Remove left/right constraint from suppression

[wasm] Disable legacy interop measurements (#81027)

Temporarily as they currently don't work

JIT: Allow forwarding field accesses off of implicit byrefs (#80852)

The JIT currently allows forwarding implicit byrefs at their last uses
to calls, but only if the full implicit byref is used. This change
allows the JIT to forward any such access off of an implicit byref
parameter.

NativeAOT: Optimize static fields of gc types (#80969)

Co-authored-by: SingleAccretion <62474226+SingleAccretion@users.noreply.github.com>

Fix HttpStress & SslStress (#80713)

Update versions to 8.0.

Fixed array delete in debugger.cpp (#79638)

m_pOffsetToHandlerInfo is array and should be delete via `delete[]`

JIT: Remove "promoted struct death vars" map (#80501)

Repurpose the multireg var death flags as general purpose "field death"
flags instead.

Eliminate `ToArray` bounds checks (#81001)

* Remove `RangeIterator.ToArray` bounds check

https://github.com/dotnet/runtime/pull/80633#discussion_r1082720454

* Eliminate additional `ToArray` bounds checks

Change how a task stress test is disabled (#80981)

Update StaticFilterOptionsMonitor.cs (#80959)

Fixes https://github.com/dotnet/runtime/issues/80902

JIT: reset max bbnum after failed inline attempt (#80958)

And if we renumber blocks in the inlinee's portion of the flow graph,
start renumbering from the max bbnum before the inline attempt, rather
than the current maximum.

More prep work in anticipation of moving pred list building to happen
before (or the root) and during (for inlinees) inlining.

Fix shared lib linking with lld-15 (#80993)

Don't look at custom attributes for field constants (#80991)

This should not be reachable.

Fold `static readonly VectorX` to GT_CNS_VEC (#80670)

Co-authored-by: Tanner Gooding <tagoo@outlook.com>

[wasm] Set `TrimMode=full` for tests (#80944)

[mono][wasm] Bundle assemblies as WebCIL (#79416)

Define a new container format for .NET assemblies that looks less like a Windows PE file. Use it for bundling assemblies in wasm projects.

* Implement WebCIL loader

  It will try to look for WebCIL formatted images instread of normal .dll files

* Checkpoint works on wasm sample; add design doc

* Push .dll->.webcil probing lower in the bundle logic

* Also convert satellite assemblies and implement satellite matching

* [wasm] don't leak .webcil image names to the debugger

   In particular this will make source and breakpoint URLs look like `dotnet://foo.dll/Foo.cs` which means that grabbing PDBs via source link will work, etc.

* Add PE DebugTableDirectory to webcil

   This is used to retrieve the PPDB data and/or the PDB checksum from an image.

   Refactor mono_has_pdb_checksum to support webcil in addition to PE images

* Implement a WebcilReader for BorwserDebugProxy like PEReader

  This needs some improvements:
   - add support for reading CodeView and EmbeddedPDB data
   - copy/paste less from the WebcilWriter task
   - copy/paste less from PEReader (will require moving WebcilReader to SRM)

* [debug] Match bundled pdbs if we're looking up .webcil files

  The pdbs are registered by wasm with a notional .dll filename. if the debugger does a lookup using a .webcil name instead, allow the match

* Adjust debug directory entries when writing webcil files

   the PE COFF debug directory entries contain a 'pointer' field which is an offset from the start of the file.

   When writing the webcil file, the header is typically smaller than a PE file, so the offsets are wrong.  Adjust the offsets by the size of the file.

   We assume (and assert) the debug directory entries actually point at some PE COFF sections in the PE file (as opposed to somewhere past the end of the known PE data).

   When writing, we initially just copy all the sections directly, then seek to where the debug directory entries are, and overwrite them with updated entries that have the correct 'pointer'

* Fix bug in WebcilWriter

   Stream.CopyTo takes a buffer size, not the number of bytes to copy.

* bugfix: the debug directory is at pe_debug_rva not at the CLI header

* skip debug fixups if there's no debug directory

* WebcilReader: implement CodeView and Emebedded PPDB support

* [WBT] Add UseWebcil option (default to true)

* rename WebcilWriter -> WebcilConverter [NFC]

* fixup AssemblyLoadedEventTest

* hack: no extension on assembly for breakpoint

* pass normal .dll name for MainAssemblyName in config

   let the runtime deal with it - bundle matching will resolve it to the .webcil file

* Wasm.Debugger.Tests: give CI 10 more minutes

* Add Microsoft.NET.WebAssembly.Webcil assembly project

   Mark it as shipping, but not shipping a nuget package.

   The idea is that it will be shipped along with the WasmAppBuilder msbuild task, and with the BrowserDebugProxy tool.

* Move WebcilConverter to Microsoft.NET.WebAssembly.Webcil

* Move WebcilReader to Microsoft.NET.WebAssembly.Webcil

   delete the duplicated utility classes

* make the webcil magic and version longer

* Code style improvements from review

* Improve some exception messages, when possible

* Suggestings from code review

* Add WasmEnableWebcil msbuild property.  Off by default

* Build non-wasm runtimes without .webcil support

* Run WBT twice: with and without webcil

   This is a total of 4 runs: with and without workloads x with and without webcil

* do the cartesian product correctly in msbuild

* also add webcil to template projects

* environment variable has to be non-null and "true"

   We set it to "false" sometimes

* Fix wasm work items

   They should be the same whether or not webcil is used.  Just the WorkloadItemPrefix should be used to change the name.

* Update src/libraries/sendtohelix-wasm.targets

* PInvokeTableGeneratorTests: don't try to use the net472 WasmAppBuilder

   Look for the default target framework subdirectory under the tasks directory in the runtime pack when trying to find the tasks dll. In particular don't try to load the net472 version on modern .NET

* PInvokeTableGeneratorTests: Add more diagnostic output if tasksDir is not found

* simplify prefix comparison in bundled_assembly_match

* WasmAppBuilder improve logging

   Just emit a single Normal importance message about webcil; details as Low importance.

* Add missing using

Co-authored-by: Ankit Jain <radical@gmail.com>
Co-authored-by: Larry Ewing <lewing@microsoft.com>

[loader] Set status on success (#80949)

Emebedders that call `mono_assembly_load_from_full` may observe a
non-NULL return value and an uninitialized MonoImageOpenStatus, which
may lead to incorrect diagnostics in code like:

```
MonoImageOpenStatus status;
MonoAssembly *assembly = mono_assembly_load_from_full (image, name,
status, refonly);
if (!assembly || status != MONO_IMAGE_OK) {
   fprintf(stderr, "Failure due to: %s\n", mono_image_strerror (status));
   abort();
}
```
Which will print `Failure due to: Internal error`

Addresses https://github.com/xamarin/xamarin-android/issues/7658

JIT: fix jit stress failures in `fgSetBlockOrder` (#80975)

Defer asserting that bbNums are ordered until we know if we're optimizing
(and hence have computed dominators).

Fixes #80974.

Update dependencies from https://github.com/dotnet/roslyn build 20230120.10 (#80971)

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23069.10 -> To Version 4.6.0-1.23070.10

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

[mono][aot] Extend Mono.ValueTuple to 7 arguments. (#80972)

This fixes a crash in the System.Text.RegularExpression tests on wasm+aot.

[mono][interp] Ensure call offset is greater than offset of other active calls (#80728)

* Add deferred call resolution for active calls. The base offset of the call must be greater than the offset of any argument of other active call args. The base offset is computed as max offset of all call offsets on which the call depends. Stack ensures that all call offsets on which the call depends are calculated before the call in question, by deferring calls from the last to the first one.

* Move call_args to InterpCallInfo struct. Allocate InterpCallInfo per call instruction.

NativeAOT: avoid helper calls for nongc static fields (#79709)

Co-authored-by: SingleAccretion <62474226+SingleAccretion@users.noreply.github.com>

Add suppression for CS8500 to test reflection/regression/github_25697 (#80964)

Handle Span<T>.Length and ReadOnlySpan<T>.Length as intrinsic (#80932)

Set change the default trimmode here too (#80940)

[wasm] Guard leading/trailing zero count (#80913)

Be more strict and allow only 32 and 64 bit integers

Final removal of uses of Windows wide char printing (#80758)

* Remove %S and %ls where possible

* Remove all non-Windows uses of %S.
Replace all %ls with %S for consistency.

Fold some expressions early in importer (#80888)

Make GetAssemblyName test more robust (#80878)

[main] Update dependencies from dotnet/roslyn (#80850)

* Update dependencies from https://github.com/dotnet/roslyn build 20230119.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23067.16 -> To Version 4.6.0-1.23069.1

* Update dependencies from https://github.com/dotnet/roslyn build 20230119.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23067.16 -> To Version 4.6.0-1.23069.10

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

Remove TEST_ProcessInfoBeforeAndAfterSuspension (#80889)

Strip all symbols on macOS as well (#80515)

* Strip all symbols on macOS as well

* Revert functions.cmake

JIT: have inliner use fgSplitBlockAfterStatement (#80928)

In anticipation of moving pred list building before inlining,
use the common block split utility instead of custom code. This does a
better job of propagating IL offsets, and automatically handles
pred list maintenance.

Contributes to #80193.