JIT: rework phase objects so we can use them more widely (#31808)

Next part of #2109.

Add support for phases that are simply compiler methods or lambdas. These are
not used yet -- the plan is to introduce new phases gradually, cleaning up
redundant checking and dumping along the way. This will happen in subsequent
changes.

Remove a bit of now-redundant post-phase dumping and checking from lower.

Add the active phase to the assertion text, so we have some context.

[interp] Make call_vararg non-recursive. (#31722)

[interp] Remove recursion from call_vararg.

This contributes to https://github.com/mono/mono/issues/18646 but is not enough to fix it.

You also have to fix newobj_fast, but that is apparently a bit difficult.

This is a reduced form of https://github.com/mono/mono/pull/18670, i.e. punting newobj_fast for now.

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Use common CLR_CMAKE_* variables in more places (#31659)

* Use common CLR_CMAKE_* variables in more places

* Remove obsolete paragraph from README

Update SkipOnCoreClr in RunContinueWithStressTestsNoState test (#31826)

Fix arm64singlestepper #ifdef (#31776)

[runtime] Report crash when receiving crashing signals (#2052)

The default handlers for SIGTRAP, SIGSYS on macOS and Linux crash the process, so we now have Mono report this.

This change also changes the presentation of the signal name on the stacktrace report dumped on error (from e.g. "SIGSEGV" to "segv").

Co-authored-by: Alexis Christoforides <alexis@thenull.net>

[interp] Stack corruption running full AOT interpreter using ValueTuple. (#31790)

Calling a method compiled using full AOT from interpreter can lead to stack corruption if signature include a ValueTuple without fields on compilers where MONO_ZERO_LEN_ARRAY is not 0 (msvc, clang). This is due to doing memset on memory allocated using MONO_SIZEOF_GENERIC_INST but
then memset using sizeof (MonoGenericInst). Fix makes sure we allocate and memset the same size of memory since otherwise it will lead to an overwrite of allocated memory when type_argc is 0.

The repro is a little tricky since the called method needs to be in a full AOT assembly and then called from an interpreted assembly:

If this is included in a full AOT:ed assembly.

```
public class StackOverwrite
{
        [MethodImpl(MethodImplOptions.NoInlining)]
        static public ValueTuple trashStack()
        {
            return ValueTuple.Create();
        }
}
```

and this is run under interpreter using --ful-aot-interp:

```
class Program
{
        static int Main(string[] args)
        {
            var test = StackOverwrite.trashStack();
            return 0;
        }
}
```

It will corrupt the stack before applying the fix.

Thanks @buggeststar for bring the issue to our attention and assisting narrow down repro on how to trigger the issues.

Co-authored-by: Johan Lorensson <lateralusx.github@gmail.com>

Remove old transport packages from CoreCLR and CoreFX. (#5523)

* Remove old transport packages from CoreCLR and CoreFX.

* Remove a lot of now unused goo from the coreclr packaging build.

* Remove unused Microsoft.Windows.Compatibility tests.

* Remove unused frameworkPackage.targets

Make more Regex loops atomic automatically (#31738)

* Change more Regex loops inside of atomic nodes to be atomic

* Allow Regex loops that end other loops to be made atomic

For example, given an expression like "(abcd*)*e", the inner d* loop will now be converted to be atomic, e.g. "(abc(?>d*))*e)".  We're careful not to convert an expression like "(abca*)*e)", where the inner loop has no overlap with what comes immediately after the loop but does with the start of the loop.

I also fixed what appears to be a long-standing bug in how character classes are canonicalized.  An attempt was made to avoid running the canonlicalization routine, but we almost always need to do it, and we were not doing it in some cases, resulting in character classes that were correct but not as optimized as they should have been, e.g. two abutting ranges when one would suffice, leading to poorer code gen.  It's part of this PR because I caught when changing the reduction tests to validate the full code tree including strings rather than just the codes list.

* Address PR feedback

And add more comments, more code coverage, more runtime asserts, etc.

I also noticed we weren't reducing Setloopatomic and Setlazy into {Not}oneloopatomic and [Not}onelazy, so I fixed that.

And I changed ReduceAtomic to actually remove unnecessary intermediate atomic nodes.

Tweak Regex compiled code and add more tests (#31730)

* Fix override visibility on generated Regex.CompileToAssembly assembly

* Add some tests based on AT&T's regex test suite

Fix host compiler when cross OS DAC compiling (#31775)

* Fix duplicate type definition MINIDUMP_TYPE

The Windows headers define MINIDUMP_TYPE. Do not define when using a
Windows HOST.

* Define DLLEXPORT macro as empty on Windows hosts

* Prefer HOST_WINDOWS to !HOST_UNIX

[Arm64] Implement simple SIMD operations (#1601)

* Update "And" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "AndNot" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Rename "AndNot" to "BitwiseClear" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Add "Divide" and "DivideScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Add "Max" and "MaxScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Add "Min" and "MinScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Add "Multiply" and "MultiplyScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Add "Negate" and "NegateScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "Not" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "Or" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "OrNot" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "Sqrt" and "SqrtScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "Subtract" and "SubtractScalar" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update "Xor" in AdvSimd.cs AdvSimd.PlatformNotSupported.cs

* Update System.Runtime.Intrinsics.Experimental.cs

* Add "And" in AdvSimd in hwintrinsiclistarm64.h

* Add "AndNot" in AdvSimd in hwintrinsiclistarm64.h

* Add "DivideScalar" in AdvSimd in hwintrinsiclistarm64.h

* Add "Divide" in AdvSimd_Arm64 in hwintrinsiclistarm64.h

* Add "Max" and "MaxScalar" in hwintrinsiclistarm64.h

* Add "Min" and "MinScalar" in hwintrinsiclistarm64.h

* Add "Multiply" and "MultiplyScalar" in hwintrinsiclistarm64.h

* Add "Negate" and "NegateScalar" in hwintrinsiclistarm64.h

* Add "Not" in AdvSimd in hwintrinsiclistarm64.h

* Add "Or" in AdvSimd in hwintrinsiclistarm64.h

* Add "OrNot" in AdvSimd in hwintrinsiclistarm64.h

* Add "Sqrt" and "SqrtScalar" in hwintrinsiclistarm64.h

* Add "Subtract" and "SubtractScalar" in hwintrinsiclistarm64.h

* Add "Xor" in AdvSimd in hwintrinsiclistarm64.h

* Add "And" in GenerateTests.csx

* Add "BitwiseClear" in GenerateTests.csx

* Add "Divide" and "DivideScalar" in GenerateTests.csx

* Add "Max" and "MaxScalar" in GenerateTests.csx

* Add "Min" and "MinScalar" in GenerateTests.csx

* Add "Multiply" and "MultiplyScalar" in GenerateTests.csx

* Add "Negate" and "NegateScalar" in GenerateTests.csx

* Add "Not" in GenerateTests.csx

* Add "Or" in GenerateTests.csx

* Add "OrNot" in GenerateTests.csx

* Add "Sqrt" and "SqrtScalar" in GenerateTests.csx

* Add "Subtract" and "SubtractScalar" in GenerateTests.csx

* Add "Xor" in GenerateTests.csx

* Update "ReverseElementBits" in GenerateTests.csx

* Update Helpers.cs Helpers.tt

* Update AdvSimd/ AdvSimd.Arm64/

[mono] Add mono_crash to the gitignore (#31815)

Use better names for Regex DynamicMethods to help when profiling (#31817)

Currently, names in the profiler just show up as "FindFirstChar37" or "Go21", and it's difficult to know with which regex it's associated.

Add cross OS support for MAKEDLLNAME (#31746)

* Add MAKE_TARGET_DLLNAME

* Add  TARGET_MAIN_CLR_DLL_NAME_[WA]

* Use TARGET_MAIN_CLR_DLL_NAME_[AW] for DAC

* Remove MSCOREE_SHIM_[WA]

Make Crossgen2 a self-contained package (#31741)

* Make Crossgen2 a self-contained package

* Deleting the crossgen2 packaging project from the coreclr sub-repo

The packaging project for the package we ship is under the installer sub-repo

allocate SocketAsyncEngine less frequenty to reduce the number of epoll_wait threads (#2346)

Fix a consistency issue with command line parsing of file names (use FileInfo instead of string) (#2033)

* Fix a consistency issue with command line parsing of file names (use FileInfo instead of string)

Removing unnecessary uses of `unreached` from hwintrinsic importation (#2298)

Delete System.Data.SqlClient package (#2275)

* Delete System.Data.SqlClient package

* Delete unnecessary SqlClient references

* Unify System.Data.SqlClient package reference

* Add System.Data facade test

* Add System.Data.SqlClient references
- Add System.Data.SqlClient package references for tests that need it
- Use System.Data.SqlClient reference assembly to build the System.Data shim
- Disable ILLinker for the shims since it does not work well with incremental builds and it is not necessary anyway

* Disable tests on Mono

Fixes https://github.com/dotnet/corefx/issues/40846

Revert "Update ILLinkTasksVersion dependency (#2334)" (#31807)

This reverts commit a05c599e0c9ba113902ba59746e041f1513cc339.

Update delegate return value to nullable (#31781)

Disable linux unwinder on Windows (#31777)

Add EMPTY_BASES_DECL (#26980)

* Add EMPTY_BASES_DECL

Add EMPTY_BASES_DECL to use MSVC __declspec(empty_bases).
This declaration tells MSVC to collapse empty base classes to
consume no space, as opposed to its default layout algoritihm
which requires base classes to consume at least one byte (in
multiple inheritance situations).

Mark a few classes which can contain empy base classes and may cause
layout differences between Windows and Clang

* Fix MapSHash deserialization

Add config knob to the gc to control its concept of what total physical memory is (#31767)

- The GC relies on the VM to compute this number normally, but the runtime uses heuristics which may be undesirable for the customer scenario (especially inside of a Windows Job object)
- This knob provides a way for the customer to override the heuristics

Account for UpperVector saves in minRegs (#31773)

When we use a jitStressRegs mode that limits the number of registers, we need to account for the UpperVectorSave.

Fix #31727

[master] Update dependencies from dotnet/arcade Microsoft/vstest (#19613)

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20080.4
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20080.4
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20080.4
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20080.4
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20080.4
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20080.4
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20080.4
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20080.4
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20080.4
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20080.4
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20080.4
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20080.4
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20080.4
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20080.4

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20081.7
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20081.7
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20081.7
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20081.7
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20081.7
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20081.7
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20081.7
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20081.7
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20081.7
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20081.7
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20081.7
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20081.7
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20081.7
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20081.7

* Update dependencies from https://github.com/microsoft/vstest build 20200131-02

- Microsoft.NET.Test.Sdk - 16.5.0-preview-20200131-02

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20101.2
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20101.2
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20101.2
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20101.2
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20101.2
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20101.2
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20101.2
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20101.2
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20101.2
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20101.2
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20101.2
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20101.2
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20101.2
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20101.2

* Rename RuntimeStressTestModes

* Update dependencies from https://github.com/microsoft/vstest build 20200203-01

- Microsoft.NET.Test.Sdk - 16.5.0-preview-20200203-01

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20103.5
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20103.5
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20103.5
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20103.5
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20103.5
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20103.5
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20103.5
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20103.5
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20103.5
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20103.5
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20103.5
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20103.5
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20103.5
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20103.5

* Update SkipOnCoreClrAttributes to new parameters

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20104.2
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20104.2
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20104.2
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20104.2
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20104.2
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20104.2
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20104.2
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20104.2
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20104.2
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20104.2
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20104.2
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20104.2
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20104.2
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20104.2

* Update dependencies from https://github.com/microsoft/vstest build 20200205-01

- Microsoft.NET.Test.Sdk - 16.5.0

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

Change RegexCompiler to special-case sets of just 2 or 3 chars (#31734)

* Change RegexCompiler to special-case sets of just 2 or 3 chars

* Apply suggestions from code review

Co-Authored-By: Dan Moseley <danmose@microsoft.com>
Co-authored-by: Dan Moseley <danmose@microsoft.com>

Disable RunContinueWithStressTestsNoState test on checked coreclr (#31789)

Avoid keeping runtext alive in Regex cache (#31736)

Null out any references to the string in case the Regex object is kept alive in a cache and the string object is very large.

Use HashCode.Combine directly (#31784)

Reduce the number of platforms that run JIT stress pipelines (#31750)

* Reduce the number of platforms that run JIT stress pipelines

The current set of platforms is overkill for JIT stress purposes.
This should reduce resource consumption and improve reliability.

Fixes #2100

Add JSON and other area CODEOWNERS (#31772)

* Add JSON and other area CODEOWNERS

* Update json owners.

[jit] Avoid calling mono_aot_direct_icalls_enabled_for_method () in non-aot mode, it might be a stub if aot is disabled. (#31754)

Co-authored-by: Zoltan Varga <vargaz@gmail.com>

Fix intermittent crash in comhost tests (#31762)

Seal CompareInfo and TextInfo (#31761)

Minor improvements to Regex.Replace overhead (#31731)

* Avoid some SegmentBuilder overhead with empty Regex.Replace replacements

* Slightly reduce Regex.Replace overhead

Reduce PAL DAC exports (#31749)

Fix check.* cross compilation linker errors (#31751)

Remove GetRecycleMemoryInfo from DAC builds (#31748)

Managed Renames for consistency (#28819)

* Managed Renames for consistency

Rename managed conditional compilation macros to match CoreCLR native concepts

    BIT64                   -> TARGET_64BIT
    BIT32                   -> TARGET_32BIT
    FEATURE_PAL             -> TARGET_UNIX
    PLATFORM_UNIX           -> TARGET_UNIX
    PLATFORM_WINDOWS        -> TARGET_WINDOWS
    PLATFORM_OSX            -> TARGET_OSX

Remove obsolete logic and obsolete defines: FeaturePal

* Simplify !TARGET_* to equivalent TARGET_*

* Remove stale SILVERLIGHT managed code

* Review feedback other

* Fix mono build break

* Fix System.Drawing.Imaging.ImageFormat.FindEncoding

* Add PAL Bridge comment

* Move FeatureCoreCLR per freedback

Don't block specific generic types in field marshalling scenar… (#31679)

Don't block specific generic types in field marshalling scenarios.

Merge pull request #31759 from DrewScoggins/RealArchFixup

Set default arch to match arcade and pass arch to ci_setup

Fix unused variable warning (#31747)

Remove duplicate architecture argument

Set default arch to match arcade and pass arch to ci_setup

Use PTRACE_GETREGSET for any arch in createdump (#22388)

Also ignore PTRACE_GETREGSET (NT_FPREGSET) failures for arm processes on
aarch64 kernels.

Disable ThrowStatementDoesNotResetExceptionStackLineOtherMethod in arm64 (#31735)

[mini] iltests.exe depends on TestDriver.dll (#3857)

Co-authored-by: Bernhard Urban-Forster <bernhard.urban@xamarin.com>

Updated composite R2R format doc (#2221)

I have resurrected my preliminary composite R2R format doc,
I rebased it to include some recent changes. For the process
itself, I assume I should first implement the self-contained
composite R2R format with standalone MSIL metadata as that's
the simplest scenario. After that I'll follow up w.r.t.
shared framework compilation and other scenarios (embedded MSIL,
single exe).

Thanks

Tomas

Fix logic to disable mscordbi build (#31745)

Reuse positive offset blocks in Regex Boyer-Moore (#31733)

* Remove unnecessary local in compiled Regex Boyer-Moore

* Reuse positive offset blocks in Regex Boyer-Moore

* Update src/libraries/System.Text.RegularExpressions/src/System/Text/RegularExpressions/RegexCompiler.cs

DeadEnding Microsoft.CSharp and Bumping Version for System.Runtime.CompilerServices.Unsafe (#2354)

* Dead Ending Microsoft.CSharp Package and Bumping the leftout assembly versions to 5.0.0.0 (#2264)

* deadending Microsoft.Csharp Package and bumping the assembly versions

* retargeting and clean up

* Update Microsoft.CSharp.csproj

* fixing the build

* reference -> projectreference

* adding ignored reference

* removing package references and making R2rDump bin place to a specifc output directory

* remove package reference from R2r dump as well

* adding comment and targeting netstandard2.0

* adding back package references for netstandard lib build

* adding back baseline version for consistency

Fix The Resource File Name Change (#31723)

[interp]Factor out common code. (#31653)

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Merge pull request #31728 from DrewScoggins/ArchFixup

Fix the way we pass the arch to the setup script

Remove unneeded type SecureStringHasher (#31707)

string.GetHashCode is already randomized on .NET Core, no need for a custom hashing routine

Fix the way we pass the arch to the setup script

Merge pull request #2393 from EgorBo/fix-mono-sample

Add RunSample target to mono.proj

Optimize `box + isinst + unbox.any` as `nop` (#1817)

Extend box pattern match to cover `box + isinst + unbox.any` .

Bump version to 6.13 (#31715)

Co-authored-by: Jo Shields <directhex@apebox.org>

Libraries code should not mutate string instances (#31700)

Update src/mono/mono.proj

Co-Authored-By: Alexander Köplinger <alex.koeplinger@outlook.com>

Update src/mono/mono.proj

Co-Authored-By: Alexander Köplinger <alex.koeplinger@outlook.com>

Remove AppDomain usage from stack walk and related places (#31708)

This is yet another cleanup removal of the AppDomain.

Address feedback

[interp] Remove goto for code_type unknown | interp | compiled. (#31667)

[interp] Remove goto for code_type unknown | interp | compiled.

The goto kinda implies a loop, but it is really just handling one case
before the other, but the source misorders the checks, perhaps for perf,
which maybe, can be done equivalently with G_UNLIKELY, and also remove
some possible extra memory reads.

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Implement Socket.DuplicateAndClose() for Windows (#1858)

* reimplement socket duplication, fix #1760

* additional cleanup

* ungroup PNSE tests, no RemoteExecutor timeout

* address review findings

* fix path for Interop.WSADuplicateSocket.cs

* remove swallowing in Socket(SocketInformation)

* review suggestions

* make sure duplicate socket is not inheritable

* remove debug code leftover

* harden SocketPal.CreateSocket()

* blittable WSAPROTOCOL_INFOW

* additional naming fixes,

default System.Net.Sockets.sln configurations to Windows_NT again

* disposal on errors, improve comments, more tests

* nits

* handle GetSockName() error

Update ILLinkTasksVersion dependency (#2334)

Clean up

When the interpreter does recurse, use a different function (#31635)

for each call site, so you can tell from a WebAssembly stack
which case it is. i.e. with function names but no line numbers.

With this, we can see that call_varargs contributes to the problem
here: https://github.com/mono/mono/issues/18646

Also, do not inline the interpreter, it probably/hopefully never was.

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Don't require Checked config for coreclr tests

[runtime] Fix the --enable-minimal=aot build. (#31684)

Fixes https://github.com/mono/mono/issues/18675.

Remove remaining libraries references to string.Copy (#31706)

Revert "Revert "Optimize typeof(T1).IsAssignableFrom(typeof(T2))" (#31643)" (#31705)

This reverts commit 729276d30fbafdc28cd9813f74c0bb49702d775c.

Merge pull request #31699 from AndyAyersMS/Fix31665

JIT: remove assert from postorder return value optimizations

Remove unused locals in System.ServiceProcess.ServiceController (#31704)

* Remove unused locals in System.ServiceProcess.ServiceController

* Inline status variable

* Apply suggestions from code review

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

[interp]Fix the type of clause_args. (#31651)

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Fix logic in compareTypesForCast (#31681)

We weren't careful enough with `__Canon` in some cases, which lead to unsafely
returning `MustNot` when the cast outcome was not determined at jit time.

Add an extra check, update comments, and add some test cases.

Addresses the failures seen in #1195 (which was reverted).

Fix issue 1871 exception test failing (#31696)

Don't adjust the offset by 1 instead of the architecture instruction size (STACKWALK_CONTROLPC_ADJUST_OFFSET).

Allow null IV for Create{Encryptor,Decryptor}.

The IV is permitted to be null for most ciphers when in ECB
mode.

Add missing nullability annotation to string ctor (#31702)

set THUMB_CODE bit (#21180)

[aot] Avoid inflating gparams with byreflike types during generic sharing. (#31670)

Fixes https://github.com/mono/mono/issues/18676.

[Arm64] Update execution characteristics for crc32b, crc32h, facge, facgt (#8500)

* Add execution characteristics for Crc32 instructions in emitarm64.cpp

* Update execution characteristics for DV_3B instructions in emitarm64.cpp

* Add execution characteristics for DV_3D instructions in emitarm64.cpp

* Update comment in emitIns_R_R_R in emitarm64.cpp

Save stack arguments size in InlinedCallFrame.m_Datum on x86 to handle callee's popping of arguments (#26834)

* Save stack arguments size in InlinedCallFrame.m_Datum on x86 to handle callee's popping of arguments

* Move stack arguments sizes logic to Windows-only

Fix package testing run name prefix (#31690)

JIT: remove assert from postorder return value optimizations

We were trying to assert that we weren't losing side effects when folding
return value trees. But this seems to be more trouble than it is worth as
the set of side effects can also change.

Also re-enable the disabled tests, and make one of them Pri0.

Fixes #31665.

Disable two JIT tests failing with known issue tracked under #31665 (#31674)

* Disable two JIT tests failing with known issue tracked under #31665

* Address Andy's PR feedback - make the exclusion clauses global

Delete unnecessary netstandard builds (#31641)

* Delete unnecessary netstandard builds and NuGet packages of inbox components

- Delete netstandard builds and NuGet packages for System.Reflection.Emit.*
- Delete netstandard builds and NuGet packages for System.Reflection.TypeExtensions
- Delete netstandard build for System.Buffers\ref
- Delete clrcompression UAP  build support

Contributes to #2138

* Remove packages from the baseline

Merge pull request #31694 from DrewScoggins/anscoggi/FixArch

* Update Arch and add RunKind variable

* Fix up Linux script as well

Fix up Linux script as well

Add Single and Double overloads to BinaryPrimitives (#6864)

* Add Single and Double overloads to BinaryPrimitives

Fix #2365

Remove popped trees more aggressively. (#31677)

When processing CEE_POP the importer sometimes created trees
like
ASG(tmp1, expr)
followed by
COMMA(ADDR(tmp1), NOP))
and that was causing some CQ issues.

LocalAddressVisitor was then marking tmp1 as address-exposed
and we never get rid of the dead ASG(tmp1, expr). What's worse, we
then were adding a zero-initialization for tmp1 in the prolog.

The fix here is to create a NOP instead of a COMMA is there are
no side-effects in the first operand of the COMMA.

This addresses the first example in #2325.

Update Arch and add RunKind variable

Merge branch 'master' of https://github.com/dotnet/runtime

Initial Crossgen2 fixes for composite R2R support (#31652)

With initial local changes to enable composite R2R I have finally
been able to compile a CoreCLR unit test in this mode. It doesn't
yet work at runtime, these changes just fix initial compiler issues
I hit during the build:

1) Delegate constructor signature was missing some comparison logic
that ended up asserting in the node comparer in the composite
build.

2) I have removed two seeminly superfluous parameters to
CreateMethodEntrypointNodeHelper.

3) I have removed an assertion related to multi-file RVA fields
that I believe to be no longer relevant after the refactorings
to copy the RVA fields around properly.

Thanks

Tomas

Link System.Globalization.Native into coreclr (#6928)

* Link System.Globalization.Native into coreclr

* Add more paths to coreclr subset filters

* Fix linux build

* PR Feedback

* PR Feedback 2

* Fix build (pass -DREPO_ROOT_DIR only on coreclr atm), System.IO.Ports.Native needs to be built first

* Remove version header file from static libs

* Remove CORECLR_BUILD variable definition

handle GetSockName() error

[mini] Add method-triggered printing for runtime stats (#2355)

This work extends the `--stats` command to take in an optional method name. When a matching method is compiled or transformed, the stats will be printed out at that point in addition to after program execution. This is useful for products looking to measure startup time and activity, as well as for ourselves to track any loader performance regressions.

Once this lands, we should see about running it on a cadence, ideally on both desktop and Android. Additionally, more work needs to be done to the counters to expose them in a way more friendly to consumption on Android/wasm (including a callback). We should also update the website to document this additional behavior.

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>
Co-authored-by: Kyle Markland <v-kymark@microsoft.com>
Co-authored-by: Rene Garcia <v-regarc@microsoft.com>